Hello,

please provide a patch for Drupal 6.16 to activate simpletest support in 6.16.

Thanks!

Comments

Anonymous’s picture

rolodmonkey’s picture

StatusFileSize
new15.8 KB

Here you go.

rolodmonkey’s picture

Status: Active » Needs review
dave reid’s picture

Status: Needs review » Fixed

Huh? It's included as the D6-core-simpletest.patch file included with the module.

rolodmonkey’s picture

Status: Fixed » Needs review

I am new to using patches, so maybe I am misunderstanding, but I just checked both the "Recommended release" and the "Development release". In both cases, the line numbers in the file that you refer to do not line up with the correct line numbers in D6.16.

The file I uploaded should be used to replace D6-core-simpletest.patch. Should I have made a patch of the patch file instead?

dave reid’s picture

Unless there's actually a conflict, the patch command is good at catching if there's offsets in line numbers and still applying it.

rolodmonkey’s picture

Status: Needs review » Closed (won't fix)

Well then, pay no attention to my file. Especially since Tortoise CVS used the wrong line endings :(

dave reid’s picture

Yeah I've heard lots of problems with Tortoise CVS and patches. :/

jdscott’s picture

Status: Closed (won't fix) » Needs review

#2: D6-16-core-simpletest.patch queued for re-testing.

Status: Needs review » Needs work

The last submitted patch, D6-16-core-simpletest.patch, failed testing.

rsevero’s picture

Status: Needs work » Needs review
StatusFileSize
new3.95 KB
new15.1 KB

Here is a new version of the D6-core-simpletest patch file AND a meta-patch to the D6-core-simpletest patch file. Please choose your poison.

Status: Needs review » Needs work

The last submitted patch, D6-core-simpletest.patch.patch, failed testing.

rsevero’s picture

Status: Needs work » Needs review
StatusFileSize
new3.76 KB
new15.1 KB

The patch and the meta-patch: take 2.

It is expected that the patch won't apply as it is the final version of the file patch available on the module.

The meta patch on the other hand should apply cleanly now.

Status: Needs review » Needs work

The last submitted patch, D6-core-simpletest.patch.patch, failed testing.

rsevero’s picture

I've just applied the current 6.x-2.x-dev patch on Drupal 6.17 and all changes succeed with some offset.

I believe this issue can be marked as fixed. I've not done it myself to let someone with more knowledge on the subject make the final decision.

ooduque’s picture

Status: Needs work » Needs review

#13: D6-core-simpletest.patch queued for re-testing.

christian.neu’s picture

#11: D6-core-simpletest.patch queued for re-testing.

Kusco’s picture

#2: D6-16-core-simpletest.patch queued for re-testing.

sorry wrong click :( :(D

boombatower’s picture

Status: Needs review » Fixed

I updated that patch for drupal-dev should be good now.

Status: Fixed » Closed (fixed)

Automatically closed -- issue fixed for 2 weeks with no activity.