On Thu, May 6, 2010 at 8:24 AM, Stephan Assmus <superstippi@xxxxxx> wrote: > > On 2010-05-06 at 17:20:26 [+0200], bga@xxxxxxxxxxxxx wrote: >> Author: bga >> Date: 2010-05-06 17:20:26 +0200 (Thu, 06 May 2010) >> New Revision: 36654 >> Changeset: http://dev.haiku-os.org/changeset/36654/haiku >> >> Modified: >> haiku/trunk/build/jam/OptionalPackageDependencies >> Log: >> I am pretty sure this is what Scott meant. ;) > > Thanks! Scott, thou shalt compile before thou commithest! > > Best regards, > -Stephan > > You guys are quick. I committed right after it got to the creating image part. Not sure why I didn't get an error with two typos in that single line. Anyways I was looking over the order and was thinking about sorting it like so: # package dependencies # # For transitive dependencies dependants should be listed before their # dependencies. OptionalPackageDependencies APR-util : APR Expat ; OptionalPackageDependencies Beam : LibIconv LibLayout OpenSSL PCRE ; OptionalPackageDependencies BeHappy : BeBook NetSurf ; OptionalPackageDependencies Curl : OpenSSL ; OptionalPackageDependencies DevelopmentBase : CDRecord DevelopmentMin Yasm ; OptionalPackageDependencies Development : DevelopmentBase Perl ; OptionalPackageDependencies GetText : LibIconv ; OptionalPackageDependencies Git : Expat Curl OpenSSL LibIconv ; OptionalPackageDependencies ICU-devel : Development ; OptionalPackageDependencies LibLayout : DevelopmentBase ; OptionalPackageDependencies NetSurf : OpenSSL Curl LibXML2 LibIconv ; OptionalPackageDependencies OpenSSH : OpenSSL ; OptionalPackageDependencies Python : Bzip ; OptionalPackageDependencies Mercurial : Python ; OptionalPackageDependencies Subversion : APR-util Neon LibIconv LibXML2 OpenSSL SQLite ; OptionalPackageDependencies Tar : LibIconv ; OptionalPackageDependencies MandatoryPackages : Sed Tar ; OptionalPackageDependencies NetFS : UserlandFS ; OptionalPackageDependencies Pe : PCRE ; OptionalPackageDependencies Transmission : LibEvent GetText Curl OpenSSL LibIconv ; OptionalPackageDependencies Vim : LibIconv ; OptionalPackageDependencies WebPositive : Curl LibXML2 SQLite ; OptionalPackageDependencies XZ-Utils : Tar ; In the current file there are some that are before and some that are after other dependencies, this update puts them in alphabetical order, except when a package is a dependency of one below it, such as Tar going one spot before MandatoryPackages where it is first needed. Ok to commit? -scottmc