How to use the patch

[Schmatze]

Sry, for me its the first time to use a diff file.

So I copied the file into the roundcube directory and wrote:

Code Select Expand

~> patch < roundcubemail-patch-0.4.2.diff
patching file CHANGELOG
can't find file to patch at input line 25
Perhaps you should have used the -p or --strip option?
The text leading up to this was:
--------------------------
|Index: skins/default/functions.js
|===================================================================
|--- skins/default/functions.js (revision 4045)
|+++ skins/default/functions.js (working copy)
--------------------------
File to patch:

then I tried:

Code Select Expand

~> patch -p1 < roundcubemail-patch-0.4.2.diff
missing header for unified diff at line 5 of patch
can't find file to patch at input line 5
Perhaps you used the wrong -p or --strip option?
The text leading up to this was:
--------------------------
|Index: CHANGELOG
|===================================================================
|--- CHANGELOG  (revision 4045)
|+++ CHANGELOG  (working copy)
--------------------------
File to patch:

Whats wrong?

Thanks

[JohnDoh]

the command should be

Code Select Expand

patch -d [path to rc dir] -p0 < roundcubemail-patch-0.4.2.diff i think

[Schmatze]

This looks better, but there's still a little problem:

Code Select Expand

patching file program/js/app.js
Hunk #1 FAILED at 1292.
Hunk #2 FAILED at 1463.
2 out of 2 hunks FAILED -- saving rejects to file program/js/app.js.rej

Code Select Expand

cat program/js/app.js.rej
***************
*** 1292,1298 ****
        }
      }
 
-     this.http_post('utils/save-pref', '_name=collapsed_folders&_value='+urlencode(this.env.collapsed_folders));
      this.set_unread_count_display(id, false);
    };
 
--- 1292,1298 ----
        }
      }
 
+     this.http_post('save-pref', '_name=collapsed_folders&_value='+urlencode(this.env.collapsed_folders));
      this.set_unread_count_display(id, false);
    };
 
***************
*** 1463,1469 ****
      if ((found = $.inArray('subject', this.env.coltypes)) >= 0)
        this.set_env('subject_col', found);
 
-     this.http_post('utils/save-pref', { '_name':'list_cols', '_value':this.env.coltypes, '_session':'list_attrib/columns' });
    };
 
    this.check_droptarget = function(id)
--- 1463,1469 ----
      if ((found = $.inArray('subject', this.env.coltypes)) >= 0)
        this.set_env('subject_col', found);
 
+     this.http_post('save-pref', { '_name':'list_cols', '_value':this.env.coltypes, '_session':'list_attrib/columns' });
    };
 
    this.check_droptarget = function(id)

[JohnDoh]

i think you should have a file called app.js.src in the folder [rc root]/program/js/ rename the file app.js to app.js.old and then app.js.src to app.js and try the patch again

[Schmatze]

Thanks this worked now perfectly.

[drewb0y]

I got exactly the same error as the other user.
After renaming the app.js and copying app.js.src back to app.js, I run the command again and get

Code Select Expand

patching file CHANGELOG
Reversed (or previously applied) patch detected!  Assume -R? [n]

Not having used patch before I'm not sure what I should answer here.

[drewb0y]

Figured it out. I reversed the previous changes using

Code Select Expand

patch -d [path to rc dir] -p0 < roundcubemail-patch-0.4.2.diff -R

And then

Code Select Expand

patch -d [path to rc dir] -p0 < roundcubemail-patch-0.4.2.diff

Which resulted in

Code Select Expand

patching file CHANGELOG
Reversed (or previously applied) patch detected!  Assume -R? [n]
Apply anyway? [n] y
Hunk #1 succeeded at 10 with fuzz 2 (offset 9 lines).
patching file skins/default/functions.js
patching file index.php
patching file program/include/iniset.php
patching file program/include/rcube_imap_generic.php
patching file program/include/rcube_vcard.php
patching file program/js/app.js
patching file program/steps/mail/func.inc

So I guess the -R reversed everything but CHANGELOG, and then after re running the patch command all is well.

(I did back it all up first just in case)

[JohnDoh]

just wondering, you applied it against a copy of 0.4.1 right?

[ABerglund]

So does there is no patch to go against 0.4-stable? I did not upgrade to 0.4.1 before the bug was discovered, so I'll need to go from that instead.

[JohnDoh]

i understood the patch was against 0.4.1 yes but i have no tested this.