automake
[Top][All Lists]
Advanced

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

Re: autopack


From: jet
Subject: Re: autopack
Date: Mon, 12 Nov 2001 03:15:53 +0900 (JST)

I've tried autopack. 
I got a rpm for autotrace(http://autotrace.sourceforge.net)!
Really nice.

Some comments.

1. Is is hard to apply a patch. We need a common repository anyway.

   --- original patch ---
   sub handle_packager {
       if ($relative_dir eq '.') {
              &require_file ('', $FOREIGN, 'packager-helper');
                  }
                      if ($options{'package-rpm'}) {
                             $output_rules .= &file_contents ('rpm');
                                 }
    }
    --- end ---
    I don't know perl syntax but...
    1.1 $FOREIGN should be FOREIGN now.
    1.2 &require_file - > require_file?

2. limits of M4 or perl syntax

Following AP_INIT_AUTOPACK statement gets a error: unblanced "'".

    AP_INIT_AUTOPACK([Program for converting bitmaps to vector graphics],
    0,
    GPL,
    Applications/Graphics,
    Masatake YAMATO<address@hidden> with Christophe Tronche's autopack patch,
dnl                                    ^^^^^^^^^^^^^^^^^^^^
    http://autotrace.sourceforge.net,
    [AutoTrace is a program for converting bitmaps to vector graphics. The
    aim of the AutoTrace project is the development of a freely-available
    application similar to CorelTrace or Adobe Streamline. In some
    aspects it is already better. Originally being created as a plugin
    for the GIMP, AutoTrace is now a standalone program and can be
    compiled on any UNIX platform using GCC.])
    
This is a bug or a spec?

3. scripts are not handled. Apply following patch:

Index: lib/am/scripts.am
===================================================================
RCS file: /cvs/automake/automake/lib/am/scripts.am,v
retrieving revision 1.42
diff -u -r1.42 scripts.am
--- scripts.am  2001/04/09 14:23:09     1.42
+++ scripts.am  2001/11/11 17:59:22
@@ -43,6 +43,16 @@
        done
 endif %?INSTALL%
 
+## ----------------------------- ##
+## Listing what gets installed.  ##
+## ----------------------------- ##
+if %?INSTALL%
+.PHONY list-install-am: list-install-%DIR%SCRIPTS
+list-install-%DIR%%PRIMARY%:
+       @list='$(%DIR%_SCRIPTS)'; for p in $$list; do \
+               echo $(DESTDIR)$(%NDIR%dir)/$$p; \
+       done 1>&2
+endif %?INSTALL%
 
 ## -------------- ##
 ## Uninstalling.  ##

4. epm
We should provide epm target, too.




reply via email to

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