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

kickstart:site doesn't work properly #3217

Closed
jonnitto opened this issue Dec 1, 2020 · 5 comments
Closed

kickstart:site doesn't work properly #3217

jonnitto opened this issue Dec 1, 2020 · 5 comments

Comments

@jonnitto
Copy link
Member

jonnitto commented Dec 1, 2020

After running the kickstart:site wizard, the generated package is in the Application folder, but the package doesn't get written into the root composer.json file.

Some thoughts:

  • Should the package not be written into the Sites folder or even better into the DistributionPackages?
  • The package should be included in the root composer.json
  • After running the wizard a message like Please run the setup or import the site with the flow command site:import --package-key Foo.Bar should be shown.
  • If a site package is already defined a warning should be shown
@jonnitto
Copy link
Member Author

jonnitto commented Dec 1, 2020

Wdyt, @Wyndala?

@jonnitto
Copy link
Member Author

jonnitto commented Dec 1, 2020

I've tested the behavior before PR #3089

The old implementation works the same way, so the wrong behavior is quite old.

@jonnitto
Copy link
Member Author

jonnitto commented Dec 1, 2020

Some further points: As it seems kickstart:package has the same problems, the package doesn't get added to the composer.json.

@jonnitto
Copy link
Member Author

jonnitto commented Dec 1, 2020

After a short call with @Wyndala we found out that the neos-development-distribution has a missing entry and folder (DistributionPackages) compared to the neos-base-distribution:

"repositories": {
  "distributionPackages": {
    "type": "path",
    "url": "./DistributionPackages/*"
  }
}

With this folder and entry in place, the package get's generated into the DistributionPackages folder and also added to the root composer.json

@jonnitto
Copy link
Member Author

jonnitto commented Dec 1, 2020

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

1 participant