After an evaluation, GNOME has moved from Bugzilla to GitLab. Learn more about GitLab.
No new issues can be reported in GNOME Bugzilla anymore.
To report an issue in a GNOME project, go to GNOME GitLab.
Do not go to GNOME Gitlab for: Bluefish, Doxygen, GnuCash, GStreamer, java-gnome, LDTP, NetworkManager, Tomboy.
Bug 386193 - should be able to check for GNUmakefile
should be able to check for GNUmakefile
Status: RESOLVED FIXED
Product: jhbuild
Classification: Infrastructure
Component: general
unspecified
Other All
: Normal normal
: ---
Assigned To: Jhbuild maintainers
Jhbuild QA
Depends on:
Blocks:
 
 
Reported: 2006-12-15 13:35 UTC by Rob Staudinger
Modified: 2007-01-10 00:09 UTC
See Also:
GNOME target: ---
GNOME version: ---


Attachments
Check for both Makefile and GNUmakefile, also ensure the .am actually exists (1.26 KB, patch)
2006-12-23 11:52 UTC, Marco Pesenti Gritti
none Details | Review
Add a makefile argument (2.33 KB, patch)
2007-01-09 23:01 UTC, Marco Pesenti Gritti
none Details | Review

Description Rob Staudinger 2006-12-15 13:35:16 UTC
Please describe the problem:
AbiWord is using "GNU" prefixed makefiles for historical reasons. It would be nice if jbhuild supported looking for GNUmakefile in addition to Makefile in the toplevel project directory, when pondering whether to autogen or just make.
An additional, optional parameter in the .modules file would seem reasonable, maybe something like 

<autotools id="libabiword" autogenargs="..." makefile="GNUmakefile">


Steps to reproduce:


Actual results:


Expected results:


Does this happen every time?


Other information:
Comment 1 Marco Pesenti Gritti 2006-12-23 11:51:41 UTC
I think it make sense to automatically check both makefiles naming, since make supports both. I'm going to attach a patch.
Comment 2 Marco Pesenti Gritti 2006-12-23 11:52:35 UTC
Created attachment 78826 [details] [review]
Check for both Makefile and GNUmakefile, also ensure the .am actually exists

OK to commit?
Comment 3 Marco Pesenti Gritti 2007-01-03 21:27:54 UTC
Can someone have a look to the patch please?
Comment 4 Frederic Peters 2007-01-03 21:56:08 UTC
Why do you have to ensure the .am actually exists?

I believe it will break the usage of autogen-sh, such as
  <autotools id="automake-1.4" autogen-sh="configure">
Comment 5 Marco Pesenti Gritti 2007-01-04 10:08:49 UTC
libabiword has both a GNUmakefile.am and a Makefile in the root directory, so checking for makefile only would not work.

To solve the autogen-sh issue would it be enough to also check for makefile + '.in' ?
Comment 6 Marco Pesenti Gritti 2007-01-04 10:11:51 UTC
Alternatively, as initially suggested by Robert, we can add a makefile optional parameter. Let me know what you prefer.
Comment 7 Marco Pesenti Gritti 2007-01-09 12:25:10 UTC
Frederic, what do you think?
Comment 8 Frederic Peters 2007-01-09 20:20:55 UTC
(sorry for late answer)  No strong preference here but I think a new attribute
will be more explicit.
Comment 9 Marco Pesenti Gritti 2007-01-09 23:01:51 UTC
Created attachment 79897 [details] [review]
Add a makefile argument

OK to commit?
Comment 10 Frederic Peters 2007-01-09 23:25:31 UTC
Fine by me.