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鈥檒l occasionally send you account related emails.

Already on GitHub? Sign in to your account

Enhancement: Sort packages by importance, then alphabetically #3872

Merged
merged 1 commit into from May 1, 2015

Conversation

Projects
None yet
4 participants
@localheinz
Copy link
Contributor

localheinz commented Mar 25, 2015

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 localheinz:feature/sort-packages branch 3 times, most recently from f94153f to 26c8033 Mar 25, 2015

@localheinz localheinz force-pushed the localheinz:feature/sort-packages branch from 26c8033 to 32538a2 Mar 25, 2015

@localheinz

This comment has been minimized.

Copy link
Contributor

localheinz commented Mar 31, 2015

@Seldaek

Any idea if you think this is cool for merging?

@hannesvdvreken

This comment has been minimized.

Copy link
Contributor

hannesvdvreken commented Apr 6, 2015

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

@greydnls

This comment has been minimized.

Copy link
Contributor

greydnls commented Apr 13, 2015

馃憤

@Seldaek Seldaek merged commit 32538a2 into composer:master May 1, 2015

1 check passed

continuous-integration/travis-ci/pr The Travis CI build passed
Details

Seldaek added a commit that referenced this pull request May 1, 2015

@Seldaek

This comment has been minimized.

Copy link
Member

Seldaek commented May 1, 2015

Thanks, made a few adjustments though:

06fb00b?w=1

ab02867

@localheinz localheinz deleted the localheinz:feature/sort-packages branch May 7, 2015

@localheinz

This comment has been minimized.

Copy link
Contributor

localheinz commented May 7, 2015

@Seldaek

Awesome, thanks a lot!

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