It looks like a few files were committed a day or two after The Great Git Migration which added some new $Id$ tags.

Patch coming...

Support from Acquia helps fund testing for Drupal Acquia logo

Comments

rocketeerbkw’s picture

I've attached a patch for 7.x and 8.x as there is a tiny change that makes one patch not apply to both.

rocketeerbkw’s picture

Status: Active » Needs review

forgot to set to needs review

Status: Needs review » Needs work

The last submitted patch, ctools-strip_cvs_tags-1788088-8.x.patch, failed testing.

rocketeerbkw’s picture

Version: 7.x-1.x-dev » 8.x-1.x-dev
Status: Needs work » Needs review
FileSize
1.55 KB

It was suggested I start with 8.x

Status: Needs review » Needs work

The last submitted patch, ctools-strip_cvs_tags-1788088-8.x.patch, failed testing.

xjm’s picture

Looks like this might be a problem with the branch and/or core compatibility. I'll retest the branch, since obviously this change is not going to affect tests.

xjm’s picture

Version: 8.x-1.x-dev » 7.x-1.x-dev
Status: Needs work » Needs review

Yep, branch tests are currently failing:
http://qa.drupal.org/pifr/test/274403

The patch in #4 is completely fine, so committed here:
http://drupalcode.org/project/ctools.git/commit/0c5521a6cc0d678f22ff6cd0...

Moving to 7.x-1.x and retesting that patch.

xjm’s picture

xjm’s picture

Status: Needs review » Fixed

Status: Fixed » Closed (fixed)

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