[haiku-commits] Re: r36654 - haiku/trunk/build/jam

  • From: scott mc <scottmc2@xxxxxxxxx>
  • To: haiku-commits@xxxxxxxxxxxxx
  • Date: Thu, 6 May 2010 08:40:39 -0700

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

Other related posts: