<!DOCTYPE HTML PUBLIC "-//W3C//DTD HTML 4.0 TRANSITIONAL//EN">
<HTML>
<HEAD>
  <META HTTP-EQUIV="Content-Type" CONTENT="text/html; CHARSET=UTF-8">
  <META NAME="GENERATOR" CONTENT="GtkHTML/3.30.3">
</HEAD>
<BODY>
Hi Jonas
<BLOCKQUOTE TYPE=CITE>
<PRE>
A few comments on your recent changes:


Arrrgh! Please be careful to commit each single logical code change 
separately, with a concise commit message - so we are best able to see 
via git changelog what has actually happened with the code. Better with 
one commit too many than one too few!  You (again!) committed 
autogenerated autotools files together with hand-coded changes, and 
changes with more invasive changes than the corresponding commit 
messages indicate!
</PRE>
</BLOCKQUOTE>
Oups Sorry I just thought this was one logical change. I mean adding a script AND including it in Makefile.am.<BR>
I 'll be more cautious in future<BR>
<BR>
<BLOCKQUOTE TYPE=CITE>
<PRE>
Please avoid ifdef contruct!  I wrote bfore about this: it should 
generally be avoided as it causes order of snippet inclusion to matter, 
which is bad.  If you do not understand what I mean, then say so, and I 
can try clarify.  If you disagree then say so, and we can discuss it.  
Just being silent about my commenting on this while ignoring it, is 
annoying!
</PRE>
</BLOCKQUOTE>
<PRE>
Oups sorry again! I don't disagree and do understand. I just did it quite fast and changed it in last commit
</PRE>
<BR>
<BLOCKQUOTE TYPE=CITE>
<PRE>
&gt;I just have a little concern : the messages are drawn in too many 
&gt;output since I do the tests with shell if and everything is echoed.
&gt;I could put all those test and unpacking in a separate shell script and 
&gt;echo only what I need. Or do you prefer implementation only in 
&gt;makefile?

I prefer using make $(warning ) and $(error ).
</PRE>
</BLOCKQUOTE>
<BR>
<BLOCKQUOTE TYPE=CITE>
<PRE>
Alternatively you can see utils.mk for an example of how to emit 
pseudo-code while silencing more convoluted actual code including 
shell-based error message.  But I don't recommend that - better to 
simplify by using make instead of shell.
</PRE>
</BLOCKQUOTE>
I did much better. No pollution output with only make instructions and target-time var expansion .<BR>
I had to make some odd constructions to test shell commands return code (like $(shell if cmd &gt; /dev/null 2&gt;&amp;1; then echo something; fi)) but its working fine.<BR>
<BR>
<BLOCKQUOTE TYPE=CITE>
<PRE>

Did you look into the relevancy of doing checksum of individual files 
inside the wad instead?

</PRE>
</BLOCKQUOTE>
I will soon !<BR>
<BR>
<BLOCKQUOTE TYPE=CITE>
<PRE>

  - Jonas

_______________________________________________
Build-common-hackers mailing list
<A HREF="mailto:Build-common-hackers@lists.alioth.debian.org">Build-common-hackers@lists.alioth.debian.org</A>
<A HREF="http://lists.alioth.debian.org/mailman/listinfo/build-common-hackers">http://lists.alioth.debian.org/mailman/listinfo/build-common-hackers</A>
</PRE>
</BLOCKQUOTE>
<BR>
</BODY>
</HTML>