bugLordsAWar! - Bugs: bug #45465, Please support automatic...

 
 

bug #45465: Please support automatic reconfiguration of the build system at build time (autoreconf)

Submitted by:  Markus Koschany <apo>
Submitted on:  Thu 02 Jul 2015 07:05:45 PM UTC  
 
Category: building/compilationSeverity: 3 - Normal
Item Group: 0.3.0Status: None
Privacy: PublicAssigned to: None
Open/Closed: Closed

Add a New Comment(Rich Markup)
   

You are not logged in

Please log in, so followups can be emailed to you.

 

(Jump to the original submission Jump to the original submission)

Fri 03 Jul 2015 07:55:51 PM UTC, comment #6:

welp, i bit the bullet and "fixed" it. I hope translations still work though!

Ben Asselstine <benasselstine>
Project Administrator
Fri 03 Jul 2015 07:29:16 PM UTC, comment #5:

And @LIBINTL@ needs to be removed from various Makefile.am files.

I don't know how intltool links my project to gettext.

Ben Asselstine <benasselstine>
Project Administrator
Fri 03 Jul 2015 07:10:32 PM UTC, comment #4:

Currently trying this:

commented out these two lines in configure.ac
AM_GNU_GETTEXT_VERSION([0.18.3])
AM_GNU_GETTEXT([external])

copy Makefile.in.in into place
(to be git added if this works)

autoreconf -fis

./configure

make distcheck

Ben Asselstine <benasselstine>
Project Administrator
Fri 03 Jul 2015 02:59:36 PM UTC, comment #3:

I've just checked the Debian package of Inkscape and it is affected by the same issue. When I try to run autoreconf, the build fails with

config.status:2749: error: po/Makefile.in.in was not created by intltoolize.

So LordsAWar simply inherited the issue from Inkscape. I would have to dig deeper into the autohell documentation to find a solution. I believe gettextize should be replaced with autopoint but I'm not an expert with this stuff and becoming one must wait a little while longer.

Markus Koschany <apo>
Fri 03 Jul 2015 12:52:57 AM UTC, comment #2:

Inkscape is a project I try to emulate because it has some similar dependencies as LordsAWar, and also because it's a mature and popluar project.

Anyway, they also use AM_GNU_GETTEXT alongside with IT_PROG_INITTOOL.

See it here:
http://bazaar.launchpad.net/~inkscape.dev/inkscape/trunk/view/head:/configure.ac

In their autogen.sh file they mess with po/Makefile.in.in which I think is the same problem I'm trying get around by running gettextize.

Ben Asselstine <benasselstine>
Project Administrator
Fri 03 Jul 2015 12:42:15 AM UTC, comment #1:

I too would like to use autoreconf instead of that hand-crafted autogen.sh script.

From my perspective however, it's not actually broken and works fine.

If you'd like to take a stab at fixing this, be my guest! Maybe it can be fixed simply by removing commands. Fixing bugs by removing code is nice when it happens :-)

Ben Asselstine <benasselstine>
Project Administrator
Thu 02 Jul 2015 07:05:45 PM UTC, original submission:

I'm the Debian maintainer of LordsAWar. I would prefer to reconfigure the build system at build time to ensure that it can always be built from source with newer versions of autotools.

LordsAWar provides a autogen.sh script for that purpose. However it requires user interaction due to the use of "gettextize -f".

According to https://www.gnu.org/software/gettext/manual/html_node/gettextize-Invocation.html

gettextsize should not be called automatically. The autopoint tool seems to be the right tool for that.

Normally in Debian I would use dh-autoreconf for recreating the build system.

https://packages.debian.org/sid/dh-autoreconf

I discovered another issue with configure.ac. Using IT_PROG_INTLTOOL and AM_GNU_GETTEXT side-by-side is unsupported

https://bugs.launchpad.net/intltool/+bug/654918

and causes the build to fail when I use dh-autoreconf. If I just use the provided build system, everything builds fine though. It would be great if autoreconf were supported at build time.

Regards,

Markus

Markus Koschany <apo>

 

(Note: upload size limit is set to 16384 kB, after insertion of the required escape characters.)

Attach File(s):
   
   
Comment:
   

No files currently attached

 

Depends on the following items: None found

Items that depend on this one: None found

 

Carbon-Copy List
  • -unavailable- added by benasselstine (Posted a comment)
  • -unavailable- added by apo (Submitted the item)
  •  

    Do you think this task is very important?
    If so, you can click here to add your encouragement to it.
    This task has 0 encouragements so far.

    Only logged-in users can vote.

     

    Please enter the title of George Orwell's famous dystopian book (it's a date):

     

     

    Follows 1 latest change.

    Date Changed By Updated Field Previous Value => Replaced By
    Fri 03 Jul 2015 07:55:51 PM UTCbenasselstineOpen/ClosedOpen=>Closed

    Back to the top


    Powered by Savane 3.1-cleanup1