Skip to content

Conversation

localheinz
Copy link
Contributor

This PR

  • sorts packages by importance (platform packages first, then PHP dependencies) and alphabetically

Follows #3549.

💡 Not sure if anyone but me uses that feature, but it would be a lot nicer if packages were sorted by importance (platform packages first), then followed by PHP dependencies, both sorted alphabetically.

See https://getcomposer.org/doc/02-libraries.md#platform-packages.

  • php represents the PHP version of the user, allowing you to apply constraints, e.g. >=5.4.0. To require a 64bit version of php, you can require the php-64bit package.
  • hhvm represents the version of the HHVM runtime (aka HipHop Virtual Machine) and allows you to apply a constraint, e.g., >=2.3.3.
  • ext-<name> allows you to require PHP extensions (includes core extensions). Versioning can be quite inconsistent here, so it's often a good idea to just set the constraint to *. An example of an extension package name is ext-gd.
  • lib-<name> allows constraints to be made on versions of libraries used by PHP. The following are available: curl, iconv, icu, libxml, openssl, pcre, uuid, xsl.

@localheinz localheinz force-pushed the feature/sort-packages branch 3 times, most recently from f94153f to 26c8033 Compare March 25, 2015 09:04
@localheinz localheinz force-pushed the feature/sort-packages branch from 26c8033 to 32538a2 Compare March 25, 2015 13:10
@localheinz
Copy link
Contributor Author

@Seldaek

Any idea if you think this is cool for merging?

@hannesvdvreken
Copy link
Contributor

Of course... I would always expect this with --sort-packages.

@greydnls
Copy link
Contributor

👍

@Seldaek Seldaek merged commit 32538a2 into composer:master May 1, 2015
Seldaek added a commit that referenced this pull request May 1, 2015
@Seldaek
Copy link
Member

Seldaek commented May 1, 2015

Thanks, made a few adjustments though:

06fb00b?w=1

ab02867

@localheinz localheinz deleted the feature/sort-packages branch May 7, 2015 17:12
@localheinz
Copy link
Contributor Author

@Seldaek

Awesome, thanks a lot!

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

Successfully merging this pull request may close these issues.

4 participants