Re: Probem with docdir sudirectory

[Sergio Belkin]

2011/1/20 Ralf Wildenhues <address@hidden>:
> Hello Sergio,
>
> * Sergio Belkin wrote on Thu, Jan 20, 2011 at 03:39:04PM CET:
>> I have a file into subdirectory of docdir that is not installed
>>
>> make distcheck complains saying:
>>
>> No rule to make target `doc/tests/exampleConf.txt', necesario para `all-am'.
>
> Does the file exist in either the source or the build tree?

Yes it does.

>
>> docdir = $(datadir)/doc/${PACKAGE}
>> dist_doc_DATA = doc/README doc/README.FEDORA doc/README.CENTOS
>> doc/multithreading.intro doc/intro doc/INSTALL
>> docdir_testsdir = $(docdir)/tests
>> docdir_tests_DATA = doc/tests/testUpThreadWorkSplittable.cc \
> [...]
>> doc/tests/testUpConf.cc doc/tests/exampleConf.txt \
>
>
>> installcheck-local:
>>         @echo "We will run a manual building against installed $(PACKAGE)"
>
> You mean 'build' not 'building' here.

Perhaps I do, my english is not perfect :)

>
>>         @sleep 2
>>         g++  -Wl,--as-needed -o  testUpLog $(srcdir)/tests/testUpLog.cc 
>> -I$(includedir)  -L$(libdir) -lUpTools
>
> Please don't hard-code compiler names and flags like that.  That's what
> AC_PROG_CXX is for, then use $(CXX) at least.

Good advice!

 For a portable variant of
> -Wl,--as-needed, use the lib-ignore module from gnulib.

But should I do that? I want that the user incidentally build the
check "by hand"

>
>>         @echo
>>         @echo  -e "You can run more tests, issuing:\n g++ -Wl,--as-needed -o 
>> binary_filename tests/source_filename -l UpTools"
>
> echo -e is not portable, you can use printf instead.

Thanks I will fix that

> A space between -l and the library name is AFAIK not portable.

Yes Sir, that's a typo :)

>
> Hope that helps.

Yes it does :)

>
> Cheers,
> Ralf
>



-- 
--
Sergio Belkin  http://www.sergiobelkin.com
Watch More TV http://sebelk.blogspot.com
LPIC-2 Certified