Closed (fixed)
Project:
Share
Version:
5.x-2.0-alpha5
Component:
Code
Priority:
Normal
Category:
Task
Assigned:
Unassigned
Reporter:
Created:
2 Aug 2008 at 06:58 UTC
Updated:
5 Nov 2008 at 21:21 UTC
Hi-
Is there any way you can tell me where I put the code to patch each module so they work with the share link. I am not a code pro so I could not figure out where it went.
Thanks so much for a great module
Comments
Comment #1
asak commentedThey both go at the end of the .module files.
I hate patches too...
Comment #2
venusrising commentedI know this sounds dumb but do I just place the at the bottom of each of their .module files? Is there anything I should look for so I don't bollocks it up?
Cheers!
Comment #3
greenskin commentedPlace the patch file in the same directory as it's destination module. So put the service_links patch file inside the service_links module directory, that is the directory with service_links.module and service_links.info (but not limited to these files) inside of it. Then to apply the patch follow the instructions at http://drupal.org/patch/apply.
You can also help convince these modules to include the patch code so that patching of the files is not needed.
Comment #4
venusrising commentedLooks like I will be SOL with this module as I don't know to SSH the patch.
Comment #5
greenskin commentedAnother option is to patch by hand, another words add all the lines with a + and subtract any lines with a -. Every line beginning with @@ and then a negative number, the negative number is the line to start at but in positive form. So if the number is -464 as in the patch for Service Links, you would start patching at line 464. Take note of lines not beginning with a + or - as these will help you make sure you are patching in the right spot.
Comment #6
venusrising commentedThanks for all of your time and help!
I will try it and see how it goes.
Comment #7
nmorse commentedPatched and received a rej from the Forward patch.
Hunk #1 FAILED at 1170.
1 out of 1 hunk FAILED -- saving rejects to file forward.module.rej
Comment #8
nrasmus commentedIf you're using the latest release of forward, it looks like you shouldn't need to patch.
Comment #9
greenskin commented