Re: [Olena-bugs] [Olena] #199: Stop using recursive Makefiles in tests (was: Stop using recursive Makefiles tests)

#199: Stop using recursive Makefiles in tests -----------------------+------------------------ Reporter: levill_r | Owner: Olena Team Type: task | Status: new Priority: minor | Milestone: Olena 2.1 Component: Milena | Version: 2.0 Resolution: | Keywords: -----------------------+------------------------ -- Ticket URL: <https://trac.lrde.org/olena/ticket/199#comment:3> Olena <http://olena.lrde.epita.fr> Olena, a generic and efficient image processing platform

Le 26 sept. 2013 à 14:39, Olena Trac <trac-notifier@lrde.epita.fr> a écrit :
#199: Stop using recursive Makefiles in tests -----------------------+------------------------ Reporter: levill_r | Owner: Olena Team Type: task | Status: new Priority: minor | Milestone: Olena 2.1 Component: Milena | Version: 2.0 Resolution: | Keywords: -----------------------+------------------------
It seems I can't comment directly on Trac, I hope answering here works as expected. Automake 1.14 makes things much easier than before, you should really have a look at %D% and %C%. You might also want to have a look at these patches in V2. As you can see, you can make this incremental: first use %C% and %D%, then remove a Makefile. I felt safer this way. commit 949d412f9934b448737d489bce9885b2a3c935c6 Author: Akim Demaille <akim@lrde.epita.fr> Date: Mon Sep 23 17:02:13 2013 +0200 build: remove last recursive Makefile * tests/Makefile.am: Rename as... * tests/local.mk: this. Adjust dependencies. commit 224ec2280d051bea141c90f9dad1db734f668784 Author: Akim Demaille <akim@lrde.epita.fr> Date: Mon Sep 23 14:48:10 2013 +0200 build: use Automake 1.14's %C% and %D% * configure.ac: Require 1.14. * tests/demo/local.mk, tests/rat/local.mk, tests/tafkit/local.mk, * tests/unit/local.mk: Avoid naming the current directory, using %D% and %C%.

On 26/09/2013 14:42, Akim Demaille wrote:
Le 26 sept. 2013 à 14:39, Olena Trac <trac-notifier@lrde.epita.fr> a écrit :
#199: Stop using recursive Makefiles in tests
[…]
It seems I can't comment directly on Trac, I hope answering here works as expected.
Unfortunately, your message did not make it to Trac. :( I'll quote your answer in the tickets for future references. I've updated your permissions on Trac's administrative panel : you should now be able to post comments (among other things) on the Olena Trac.
Automake 1.14 makes things much easier than before, you should really have a look at %D% and %C%. You might also want to have a look at these patches in V2. As you can see, you can make this incremental: first use %C% and %D%, then remove a Makefile. I felt safer this way.
commit 949d412f9934b448737d489bce9885b2a3c935c6 Author: Akim Demaille <akim@lrde.epita.fr> Date: Mon Sep 23 17:02:13 2013 +0200
build: remove last recursive Makefile
* tests/Makefile.am: Rename as... * tests/local.mk: this. Adjust dependencies.
commit 224ec2280d051bea141c90f9dad1db734f668784 Author: Akim Demaille <akim@lrde.epita.fr> Date: Mon Sep 23 14:48:10 2013 +0200
build: use Automake 1.14's %C% and %D%
* configure.ac: Require 1.14. * tests/demo/local.mk, tests/rat/local.mk, tests/tafkit/local.mk, * tests/unit/local.mk: Avoid naming the current directory, using %D% and %C%.
Thanks for the advice! This seems really nice. -- Roland Levillain EPITA Research and Development Laboratory (LRDE) 14-16, rue Voltaire - FR-94276 Le Kremlin-Bicêtre Cedex - France Phone: +33 1 53 14 59 45 - Fax: +33 1 53 14 59 22 - www.lrde.epita.fr
participants (3)
-
Akim Demaille
-
Olena Trac
-
Roland Levillain