Cannot download Mail_Mime from SVN repo via Include module

#2 worked great for me, thanks madmatter23!

Could someone send me the mailmime.module file with the patch applied ?
Maybe just the part that needs to be changed.

I'm trying to do the changes in the code myself but I don't understand the "@@-121,7 +126,7 @@" part
It would be really appreciated !

Thank you !

The @@-121,7 +126,7@@ part describes the starting line number and number of matching lines in both the original and pached/edited/final file:

1. Start at line 121 of the original file

2. Match seven lines of text

3. Which will become line 126 in the patched file

4. And match seven lines of text there, as well.

Thank you for helping me.

I didn't wanted to go through all that documentation cause I tought it was only a little problem and I had no need to know how to patch a module. For now.

I'm using Quickstart on VM Virtual Box.
When I patch I get this error.

Hunk #1 FAILED at 96.
Hunk #2 FAILED at 121.
2 out of 2 hunks FAILED

At first I thought it was cause my module version was 7.x-2.15 and in your message title it says Version: 7.x-1.x-dev. So I downloaded that version but still the same error.

Have an idea what I'm doing wrong ?
I would like you to tell me if the "Quickstart" solution is a good way. Cygwin better ?

Thank you !

Fixing now.

Fixed in 6.x-2.17 / 7.x-2.17 / 8.x-2.17 releases.

Thanks very much for the report and patah.

It still doesn't work with version 7.x-2.17
Any other idea for those who are not PEAR expert?
Thank you.

No wonder it's not working in version 2.17.
First the path has changed and there is a typo in the name of the file:
It's not Mail/mimeDecode.php but Mail_mimeDecode.php. The slash should be an underscore..!
Error downloading (https://raw.github.com/pear/Mail_Mime/master/Mail/mimeDecode.php): Not Found

The issue is that mimeDecode.php is still listed in the the Mail_mime array in mailmime_download_requirements(). Somehow this got left out of the commits on all branches relating to this issue.

Here are two patches.

The first patch is against 7.x-2.17.

The second patch is against 7.x-2.x-dev (the 7.x-2.x branch, which is a bit inconsistent, I'm filing another issue about a typo present in 7.x-2.x but not 7.x-2.17).
Edit: The second patch is almost identical to the patch from #2 here, so it's probably unnecessary at this point.

The first patch just removes the same line as #1821686: Failed to download mimeDecode.php, but that issue is for 6.x-2.x. 8.x-2.x needs to be patched as well.

It would be great if this could be committed.

I'm explaining to some people how to install mailmime - and since not all of them have command line access they can not

pear install -a Mail_Mime

or apply any patches either. They rely on Include module.

Thanks - Merci!

Looking back in this thread - #9/#10 -> Fixed 7.x-2.17 - something didn't go right:

mimeDecode.php is still in:

  $packages = array(
    'Mail_mime' => array(
      'local_path' => 'Mail/',
      'remote_path' => 'https://raw.github.com/pear/Mail_Mime/master/Mail/',
      'files' => array(
        'mime.php',
        'mimeDecode.php',
        'mimePart.php',
      ),

And simply needs to be commented out (or deleted):

- 'mimeDecode.php',

That's all.

@KarinG - Yes, that's what I found as well, and that's what my first patch does, the 2.x-dev branch is a bit different though. If you can test and review the patch(es) in #14 you can RTBC this issue and hopefully get us closer to a commit!

#14: mailmime-1417646-7.x-2.x-dev-14.patch
applied to HEAD

Fixed in the D7 and D6 -dev versions (give them up to 12h to be repackaged).

Thanks Cottser and KarinG!