Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Minor cleanup #311

Closed
wants to merge 35 commits into from
Closed

Minor cleanup #311

wants to merge 35 commits into from

Conversation

tinybeachthor
Copy link
Contributor

@tinybeachthor tinybeachthor commented Sep 30, 2022

After #300

#315

Set version in `composer.json` to the package version,
composer is conservative and will fail if there the
package version is not set and any dependency version
conflicts are declared.
Without composer.json in the root of a package,
composer will ignore the dependency, this is issue if
for example npm dependencies are managed via composer.
(yiisoft/yii2 2.0.46)
Without composer.json in the root of a package,
composer will ignore the dependency, this is issue if
for example npm dependencies are managed via composer.
(yiisoft/yii2 2.0.46)
package name and composer.json#name can be different
(mostly happens in private repositories)
set the composer.json#name to the package name
before including the package in other one
no network access in the pure builder,
patches will have to be handled as overrides probably
when composer is installing from source it ignores
git submodules, do the same
@DavHau DavHau added the summer-of-nix Issues/PRs opened by SoN participants label Oct 17, 2022
raboof added a commit to raboof/dream2nix that referenced this pull request Mar 18, 2023
It looks like `rootName` should be set to the *parent* name,
but currently points at the name of the dependency component.

Fixes nix-community#488, though it seems this was changed intentionally by
@tinybeachthor in nix-community#311 (specifically 156d7e0),
so this might break something else?
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
summer-of-nix Issues/PRs opened by SoN participants
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants