gnash-commit
[Top][All Lists]
Advanced

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

[Gnash-commit] [patch #7554] Autoinstalling gnash dependencies


From: Jonas 'Sortie' Termansen
Subject: [Gnash-commit] [patch #7554] Autoinstalling gnash dependencies
Date: Thu, 07 Jul 2011 22:29:12 +0000
User-agent: Mozilla/5.0 (X11; Linux x86_64; rv:5.0) Gecko/20100101 Firefox/5.0

Follow-up Comment #6, patch #7554 (project gnash):

I have now attached an updated patch that addresses a few key issues:

1) All the suggestion output is now done with macros. This macro system then
takes care of formatting the output, and setting up the list of package names
for the local system.
2) The script now installs everything in a single iteration. 

When fixing 2) I had to change a few tests in the suggestion phase. Some of
them weren't coded consistently and I may have cause some regressions. It
would be nice if you looked at my TODOs to make sure I have not
unintentionally broken some exotic system.

I use a few shell utilities to format the output (sed, fold, cut, tr) and as
far as I know they are in POSIX and part of MSYS, so their presence shouldn't
break any systems. In any case, the suggestion lines are hardly useful on any
non-GNU systems.

Since the macros are easy to tweak, it could be easy to disable some of the
suggestions based on whether you have apt, yum, or yast. I decided on keeping
them, as it shows users that we care about many platforms.

In the case of Gnash, having a static install-everything.sh script is
impossible. We simply support too many configurations, backends, frontends,
package management systems, for this to be possible without using configure.

Another thing that might need to be addressed, is that this script installs
everything from the "error", "warning", and "recommended" categories. It may
be desirable to only install from the "error" and "warning" category. But on
the other hand, if the user is really picky, they will just run the
apt/yum/yast lines anyways.

(file #23630)
    _______________________________________________________

Additional Item Attachment:

File name: 0001-Configure-generates-a-script-that-installs-everythin.patch
Size:56 KB


    _______________________________________________________

Reply to this item at:

  <http://savannah.gnu.org/patch/?7554>

_______________________________________________
  Message sent via/by Savannah
  http://savannah.gnu.org/




reply via email to

[Prev in Thread] Current Thread [Next in Thread]