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

Only include composer in apps that use it #1563

Closed
10 of 15 tasks
JedMeister opened this issue Feb 10, 2021 · 5 comments
Closed
10 of 15 tasks

Only include composer in apps that use it #1563

JedMeister opened this issue Feb 10, 2021 · 5 comments

Comments

@JedMeister
Copy link
Member

JedMeister commented Feb 10, 2021

As a streamlining/minimisation measure, we should only include composer in apps that use it. Although LAMP and LAPP should probably be exceptions to the rule.

To do that, we should remove the composer.mk from the common lamp.mk & lapp.mk and just manually add it to the Makefile of LAMP, LAPP and any appliance that uses it.

The apps that will need the changes (i.e. ones that use composer, or we decide to include it) have been noted below and tagged in this issue.

  • Remove from lamp.mk & lapp.mk common makefiles

Add to appliance Makefiles

  • lamp
  • lapp
  • bagisto
  • cakephp [note: buildcode doesn't yet use it (see CakePHP - use composer to install #1591) - but would be useful for users, so included now]
  • drupal7
  • drupal8
  • drupal9
  • ezplatform
  • laravel
  • magento
  • moodle
  • silverstripe
  • snipe-it
  • typo3
@JedMeister
Copy link
Member Author

Removed suitecrm - turns out that it wasn't actually using composer (it was in the conf script but commented out): turnkeylinux-apps/suitecrm@c41228e

@OnGle
Copy link
Member

OnGle commented Aug 3, 2021

ping @JedMeister

@JedMeister
Copy link
Member Author

@OnGle - this is another one that we'll need to double check with repos that it's been done (or not). I have update the checklist with the merged PRs noted here, but there are still some outstanding (they may have been done, but I can't confirm right now).

@JedMeister
Copy link
Member Author

JedMeister commented Nov 16, 2023

Removed snipe-it - it uses composer now anyway...

@JedMeister
Copy link
Member Author

Actually, I've just made an executive decision to close this issue....

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

3 participants