| 19 May 2023 |
@drupol:matrix.org | Checking | 20:29:32 |
@drupol:matrix.org | Do you think this would fix everything? | 20:29:52 |
Jan Tojnar | not everything but at least this thing | 20:30:06 |
@drupol:matrix.org | mmh mmh | 20:30:29 |
@drupol:matrix.org | Ok | 20:30:31 |
@drupol:matrix.org | Curious to see how you're going to patch it. | 20:30:51 |
Jan Tojnar | I was thinking resolving the path relative to repository path | 20:31:03 |
Jan Tojnar | but thinking about it more, it does not really make much sense since the repository does not have to be a path, does it? | 20:31:31 |
@drupol:matrix.org | There could be different type of repos indeed | 20:32:26 |
Jan Tojnar | I guess we could do it like napalm does and run http server | 20:33:38 |
@drupol:matrix.org | I have no clue on those things | 20:33:55 |
Jan Tojnar | Pol: regarding the composer packaging it looks like composer update --lock will require even developer dependencies to be in the repo | 21:19:31 |
Jan Tojnar | which makes sense, --no-dev should not produce partial lockfile | 21:19:58 |
@drupol:matrix.org | Mmh mmh | 21:20:27 |
Jan Tojnar | so we cannot support includeDev in the fetchComposerDeps function | 21:20:35 |
Jan Tojnar | and need to optimize the fetcher instead | 21:20:44 |
@drupol:matrix.org | But we can do `composer update --lock --no-dev` | 21:20:52 |
Jan Tojnar | we can, but it will still look for dev dependencies so that it can lock them | 21:21:18 |
Jan Tojnar | it just will not install them | 21:21:31 |
Jan Tojnar | * it just will skip installing them | 21:21:44 |
@drupol:matrix.org | Right, so, we have to parse lock files and intersect with composer.json.require then ? | 21:22:21 |
Jan Tojnar | we could have the setup hook patch the sources inside the lockfile, instead of running composer update --lock | 21:23:38 |
Jan Tojnar | and rely on the fact that the dev dependencies will not be installed with --no-dev so it would not matter that the path would be non-existent | 21:24:19 |
Jan Tojnar | or we could just optimize the fetcher so that downloading dev dependencies is not pain | 21:25:00 |
@drupol:matrix.org | I prefer to optimize the fetcher | 21:25:21 |
@drupol:matrix.org | Maybe we can just skip the package if the package is not in require section. | 21:27:53 |
Jan Tojnar | actually, maybe just using the dists would be fine for you, then | 21:27:54 |
@drupol:matrix.org | What do you mean? | 21:28:06 |
@drupol:matrix.org | How about making the changes in the composer plugin ? | 21:28:23 |
@drupol:matrix.org | Just added you as a maintainer | 21:29:25 |