Support for Drupal 7 is ending on 5 January 2025—it’s time to migrate to Drupal 10! Learn about the many benefits of Drupal 10 and find migration tools in our resource center.
This issue is about following the implementation of the code at #1293956: Add an option for reposync plugins to store tag/commit relations optionally
Comment | File | Size | Author |
---|---|---|---|
#8 | optional-label-mapping.patch | 27.56 KB | marvil07 |
#8 | interdiff.txt | 9.44 KB | marvil07 |
#4 | 0001-Issue-1985426-Implement-versioncontrol-reposync-plug.patch | 18.59 KB | marvil07 |
#1 | 0001-Issue-1985426-Implement-versioncontrol-reposync-plug.patch | 18.11 KB | marvil07 |
Comments
Comment #1
marvil07 CreditAttribution: marvil07 commentedA new version of the patch after changes related with default branch on git_default reposync plugin. This still needs work, but NR for testbot.
Comment #3
marvil07 CreditAttribution: marvil07 commentedUhm, bot is failing because it is using last release of versioncontrol instead of the dev version tarball, I added a new release on versiocontrol, but not sure how often testbot notices new releases. Let's wait a little, or if someone knows how to let bot use dev tarball please tell me :-)
Comment #4
marvil07 CreditAttribution: marvil07 commentedlet's try adding a version requirement on dependencies array at info file
Comment #6
marvil07 CreditAttribution: marvil07 commentedThere was a bug on versioncontrol about sync options, it's now fixed, and now tests are passing locally.
I also changed versioncontrol_git dependency to use versioncontrol 7.x-1.x-dev, hopefully that will make testbot choose the dev tarball, but I will wait to re-test until next rebuild of dev tarballs and until the next project dependencies rebuild, but I think the patch as it is now(comment 1 patch) will go into upstream soon.
Comment #7
marvil07 CreditAttribution: marvil07 commented#1: 0001-Issue-1985426-Implement-versioncontrol-reposync-plug.patch queued for re-testing.
Comment #8
marvil07 CreditAttribution: marvil07 commentedOk, I give up with both, it postponed the patch because a current 7.x-1.x branch here needs a patch to avoid a fatal because of signature change. I run tests locally and it worked correctly, so adding the attached patches.
Comment #9
marvil07 CreditAttribution: marvil07 commentedNow NW for D6 version.
Comment #10
marvil07 CreditAttribution: marvil07 commentedSince 6.x-2.x entered in maintenance mode, this will not make it there.