- Issue created by @phenaproxima
- 🇺🇸United States phenaproxima Massachusetts
Oh, this will also need an upstream follow-up to ensure that Composer Stager's PHP file syncer also treats its exclusions the same way rsync does. Consistency is crucial here.
- last update
over 1 year ago CI aborted - @phenaproxima opened merge request.
- last update
over 1 year ago CI aborted - last update
over 1 year ago CI aborted - last update
over 1 year ago CI aborted - last update
over 1 year ago Custom Commands Failed - Assigned to tedbow
- Status changed to Needs review
over 1 year ago 5:34pm 6 June 2023 - last update
over 1 year ago 784 pass, 5 fail - last update
over 1 year ago 784 pass, 4 fail - Issue was unassigned.
- Assigned to omkar.podey
- last update
over 1 year ago 784 pass, 5 fail - last update
over 1 year ago 784 pass, 4 fail - 🇮🇳India omkar.podey
i just reverted my change, seems like i'm missing something, wouldn't leaving the slashes in the prefix make relative paths appear to be absolute?
- Issue was unassigned.
- last update
over 1 year ago 784 pass, 5 fail - 🇮🇳India omkar.podey
the excluded paths lists looks like, most of them now have paths staring from
./
but i don't think they are being actually excluded because of the failing tests.
- 🇮🇳India omkar.podey
Maybe I shouldn't have added the
./
to the file names and only to the directories. - last update
over 1 year ago 785 pass, 4 fail - last update
over 1 year ago Custom Commands Failed - last update
over 1 year ago 789 pass, 4 fail - last update
over 1 year ago 790 pass, 3 fail - last update
over 1 year ago 792 pass, 2 fail - 🇮🇳India omkar.podey
is this going in the right direction ? or am i missing something?
- Assigned to phenaproxima
- Status changed to Needs work
over 1 year ago 11:00am 12 June 2023 - 🇧🇪Belgium wim leers Ghent 🇧🇪🇪🇺
The issue summary only talks about the
rsync
file syncer, not thephp
one.How does this affect that? AFAICT it shouldn't affect that at all, because we're not collecting a list of patterns to exclude, but a list of paths to exclude. This becomes clearer by reading
\Drupal\package_manager\PathExcluder\GitExcluder::excludeGitDirectories()
, which if we were excluding patterns wouldn't have to first find all paths to exclude: it could then just exclude.git
and be done.Isn't this just a bug in Composer Stager? It's either a bug in:
\PhpTuf\ComposerStager\Infrastructure\Service\FileSyncer\RsyncFileSyncer
? AFAICT that's just misinterpreting thePathListInterface $exclusions
it receives.- or in
\PhpTuf\ComposerStager\Infrastructure\Value\PathList\PathList::assertValidInput()
- Status changed to Postponed
over 1 year ago 1:51pm 12 June 2023 - 🇺🇸United States phenaproxima Massachusetts
Opened https://github.com/php-tuf/composer-stager/issues/183 against Composer Stager.
Assuming this gets fixed upstream (and I think it is likely that it will be), we should probably just close out this issue once we have updated to a new release that contains the bugfix. But for now, postponing on that.
- Status changed to Active
over 1 year ago 1:44pm 23 June 2023 - last update
over 1 year ago 813 pass - last update
over 1 year ago 813 pass - Assigned to tedbow
- Status changed to Needs review
over 1 year ago 2:23pm 23 June 2023 - Status changed to RTBC
over 1 year ago 5:01pm 23 June 2023 - last update
over 1 year ago 813 pass -
phenaproxima →
committed c85daa53 on 3.0.x
Issue #3365151 by phenaproxima, omkar.podey: Update Composer Stager to 2...
-
phenaproxima →
committed c85daa53 on 3.0.x
- Status changed to Fixed
over 1 year ago 5:05pm 23 June 2023 - Issue was unassigned.
Automatically closed - issue fixed for 2 weeks with no activity.