Bernard Li wrote:
> Hi Andrea:
> 
> On 12/2/07, Andrea Righi <[EMAIL PROTECTED]> wrote:
> 
>> Yes, the only drawback is that it's only needed to build PS3 packages, and it
>> wouldn't be good to introduce a new dependency for all the other 
>> architectures,
>>  but probably this is better than failing the build process for a missing
>> dependency.
> 
> At least for RPM, I only added the dependency if we're on PS3, see:
> 
> http://www.systemimager.org:8000/trac.systemimager.org/changeset/4355
> 
> Didn't really test it, but should work :-)

OK, I'll see if it's possible to do something similar with deb.

> 
> BTW, questions about this changeset:
> 
> http://svn.systemimager.org:8000/trac.systemimager.org/changeset/4357
> 
> Is the VERSION file really needed?  Perhaps we should modify simply
> the configure.ac file for version increments (less files to modify =
> less prone to errors).
> 
> Should we really check in the configure script?  It might be better to
> have the developer simply run bootstrap (or whatever tool to generate
> the configure script).
> 

Sounds good to have a single file. Anyway, the version string in the configure
is pretty useless: it appears only in the configure help and in some comments,
so I wouldn't consider it a critical issue.

IMHO the best solution is to add a simple code in autoconf/bootstrap to
automatically re-generate both the VERSION and the configure files, depening on
the version string defined in autoconf/configure.ac. In this way we could simply
remove VERSION and configure from the svn repository, since they'll be
re-generated automatically as an initial build step.

-Andrea

-------------------------------------------------------------------------
SF.Net email is sponsored by: The Future of Linux Business White Paper
from Novell.  From the desktop to the data center, Linux is going
mainstream.  Let it simplify your IT future.
http://altfarm.mediaplex.com/ad/ck/8857-50307-18918-4
_______________________________________________
sisuite-devel mailing list
[email protected]
https://lists.sourceforge.net/lists/listinfo/sisuite-devel

Reply via email to