whizard is hosted by Hepforge, IPPP Durham

Opened 15 years ago

Last modified 14 years ago

#206 closed defect

share/test/Makefile.am and/or test/Makefile.am modify $(srcdir) — at Version 11

Reported by: Juergen Reuter Owned by: ohl
Priority: P2 Milestone:
Component: configure Version: 2.0.0rc2
Severity: critical Keywords: structure
Cc:

Description (last modified by ohl)

Makefiles must respect the separation of $(srcdir) and $(builddir).

test/Makefile.am
share/test/Makefile.am 

should be properly autotoolized. If there is foo.m4, then foo.sin should not be in $(srcdir).

Change History (11)

comment:1 Changed 15 years ago by Juergen Reuter

Specifically a broken make distcheck means THAT ALL OTHER ACTIVITIES have to stop before that problem is settled. This absolutely highest priority.

comment:2 Changed 15 years ago by Juergen Reuter

Now it seems to work up to those permission errors which do not do any harm (cf. below). Problem is that I have to go exclusively via the srcdir and omit the builddir, because otherwise I have to copy files over again, which is bullshit.

/usr/bin/m4 ../../../share/test/testsuite.m4 ../../../share/test/mssmtest-ee2.m4 > ../../../share/test/mssmtest-ee2.sin
/bin/sh: ../../../share/test/mssmtest-ee2.sin: Permission denied
make[3]: [mssmtest-ee2.sin] Error 1 (ignored)
/usr/bin/m4 ../../../share/test/testsuite.m4 ../../../share/test/mssmtest-en.m4 > ../../../share/test/mssmtest-en.sin
/bin/sh: ../../../share/test/mssmtest-en.sin: Permission denied
make[3]: [mssmtest-en.sin] Error 1 (ignored)
/usr/bin/m4 ../../../share/test/testsuite.m4 ../../../share/test/mssmtest-tt.m4 > ../../../share/test/mssmtest-tt.sin
/bin/sh: ../../../share/test/mssmtest-tt.sin: Permission denied
make[3]: [mssmtest-tt.sin] Error 1 (ignored)
/usr/bin/m4 ../../../share/test/testsuite.m4 ../../../share/test/mssmtest-tn.m4 > ../../../share/test/mssmtest-tn.sin
/bin/sh: ../../../share/test/mssmtest-tn.sin: Permission denied
make[3]: [mssmtest-tn.sin] Error 1 (ignored)

comment:3 Changed 15 years ago by Juergen Reuter

Description: modified (diff)
Priority: P1P3
Severity: blockernormal
Summary: make distcheck is broken by m4 -> sin conversionRemove ugliness from share/test Makefile.am

it's working. About the rest I don't care. Now someone who knows Autotools better has to clean up the mess :-s .

comment:4 Changed 15 years ago by Juergen Reuter

Priority: P3P1

Maybe its better to address this before it affects anything else!!!

comment:5 Changed 15 years ago by Juergen Reuter

Summary: Remove ugliness from share/test Makefile.amUpdating of the .m4 files is also not working

oK, this is a show stopper, updating of the m4 files does not work. I'm done with that f*ing autotools. They are way above me.

comment:6 Changed 15 years ago by ohl

Owner: changed from ALL to ohl
Severity: normalcritical
Status: newassigned

OK, I'll try to fix that. But no promises before Monday.

comment:7 Changed 15 years ago by ohl

as of r1787 (autoreconf!), the m4 action has been moved to a branch

branches/ohl/properly-autotoolize-tests 

and the trunk compiles again.

comment:8 Changed 15 years ago by Juergen Reuter

Don't know whether this was wise. Sometimes it's better to have progress than to have the trunk compiling. Now my tests are screwed. I really hate it.

comment:9 Changed 15 years ago by Juergen Reuter

Summary: Updating of the .m4 files is also not workingStreamlining of Makefiles for .m4 -> .sin conversion

comment:10 Changed 15 years ago by Juergen Reuter

The updating of .m4 -> .sin files seems to work when the .m4 files get affected.

comment:11 Changed 15 years ago by ohl

Description: modified (diff)
Priority: P1P2
Summary: Streamlining of Makefiles for .m4 -> .sin conversionshare/test/Makefile.am and/or test/Makefile.am modify $(srcdir)

See the new description.

Don't panic. I'll fix this.

Note: See TracTickets for help on using tickets.