<div dir="ltr"><div class="gmail_extra"><div class="gmail_quote"><div><div>Hi Wookey, thanks a lot for your comments.</div></div><div><br></div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left-width:1px;border-left-color:rgb(204,204,204);border-left-style:solid;padding-left:1ex">1) Makefile and loch/Makefile (at least, maybe more) have been<br>
converted from unix to DOS linefeeds. Thus breaking the patches. Was<br>
that just an accident due to an over-enthusiastic editor? Is it going<br>
to stay like that or can we agree whether the release is<br>
DOS-throughout or unix throughout? (it's now a mix).<br></blockquote><div><br></div><div>It is an error. It should be fixed in the updated archive on the server.</div><div><br></div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left-width:1px;border-left-color:rgb(204,204,204);border-left-style:solid;padding-left:1ex">I see that the whitespace patch which fixed up a load of inconsistent<br>
tab/space breakage has also not been applied: therion-5.3.12-whitespace.patch<br></blockquote><div><br></div><div>I would like to fix it in the next release, together with all compiler warnings.</div><div> </div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left-width:1px;border-left-color:rgb(204,204,204);border-left-style:solid;padding-left:1ex">2) A couple of files are included that should not be:<br>
samples/us.stackdump<br>
thdb1d.cxx.orig<br>
thbook/etc/contents.tex (that's a generated file, right?)<br>
(this last was also in 5.3.15 so is not a regression)<br></blockquote><div><br></div><div>Again errors. With exception of contents.txt - it is regenerated every time, but I am not sure, whether it can be removed.  </div><div> </div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left-width:1px;border-left-color:rgb(204,204,204);border-left-style:solid;padding-left:1ex">3) You have indeed included several of the debian patches (much appreciated),<br>
but the upstream fixes in 20debianise-makefiles.patch have not been included:<br>
(all in 'Makefile')<br>
<br>
therion needs to be built before the samples can build:<br>
-samples: $(OUTDIR)/samples.doc/index.tex<br>
+samples: $(OUTDIR)/samples.doc/index.tex therion<br>
<br>
windres does not exist on non-windows platforms (maybe macos?)<br>
+ifeq ($(THPLATFORM),WIN32)<br>
 $(OUTDIR)/therion.res: therion.rc<br>
                        windres -i therion.rc -J rc -o $(OUTDIR)/therion.res -O coff<br>
+endif<br>
<br>
all the thTMPDIRs from the build need cleaning, not just the top one:<br>
+   perl <a href="http://makefile.pl" target="_blank">makefile.pl</a> rmdir -q thTMPDIR samples/*/thTMPDIR samples/*/*/thTMPDIR<br>
(that could be a find . -name thTMPDIR instead if you preferred a more general solution)<br>
<br>
(I couldn't see this being fixed anywhere else, but maybe it has been?)<br></blockquote><div><br></div><div>I will update this in the next release.</div><div> </div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left-width:1px;border-left-color:rgb(204,204,204);border-left-style:solid;padding-left:1ex">other patches not applied. Are there reasons why not?<br>
* 82-nolang-segfault-fix.patch  (this avoids a genuine error case of language not being defined)<br></blockquote><div><br></div><div>I believe, this error is already fixed. Are you able to reproduce it with current version of therion?</div><div> </div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left-width:1px;border-left-color:rgb(204,204,204);border-left-style:solid;padding-left:1ex">
* 90load-extensionless-files.patch (fixes bug reported on this list a while back)<br></blockquote><div><br></div><div>This needs to be done. </div><div><br></div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left-width:1px;border-left-color:rgb(204,204,204);border-left-style:solid;padding-left:1ex">
* 2 hunks of fix-compiler-warnings.patch was applied, but not the other 19. Any reason?<br></blockquote><div><br></div><div>No reason for that, will try to do it in the near future.</div><div><br></div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left-width:1px;border-left-color:rgb(204,204,204);border-left-style:solid;padding-left:1ex">
The remaining patches/hunks are debian-specific<br>
<br>
Anyway packaging now updated and 5.3.16-1 uploaded to<br>
experimental. (5.3.15-1 need to stay in unstable until the Debian<br>
Freeze for the upcoming stable release is over, so new versions go in<br>
experimental)<br>
<br>
Feedback welcome, as ever.<br></blockquote><div><br></div><div>Thanks again, S. </div></div></div></div>