[Author Prev][Author Next][Thread Prev][Thread Next][Author Index][Thread Index]

Re: GraphThing 1.3.1 released



David Symonds <dsymonds@xxxxxxxxx> (19/09/2006):
> This is a maintenance release, fixing up some of the build process and
> rolling in some 64-bit fixes and a new manpage. If you're not having
> any problems, don't bother upgrading.

Again, many thanks for your efforts! Some troubles subsist though:
 - the location of the manpage is expanded into a path including
   ${prefix}, which is not defined in GNUmakefile.in. The tiny first
   attached patch fixes that.
 - it looks like you now use a Perl script instead of a shell one, but
   you didn't rename it in src/GNUmakefile.in. See the second attached
   patch for an easy fix.

When building, a lot of warnings is(are?) displayed, complaining about
``dereferencing type-punned pointer will break strict-aliasing rules''
(I'm using gcc 4.1.2). That's not a problem to me, but I prefer telling
you that now so it won't become a major problem afterwards,
surprisingly, if the compiler behaviour is modified. (gcc is being more
and more restrictive and strict.)

Thanks for the manpage and for the 64-bits fix. I hope that some friends
will be able to test the build during the evening.

Cheers,

-- 
Cyril
--- graphthing-1.3.1~/GNUmakefile.in    2006-09-19 13:54:12.346112864 +0000
+++ graphthing-1.3.1/GNUmakefile.in     2006-09-19 13:53:02.465736296 +0000
@@ -4,6 +4,7 @@
 VERSION = @PACKAGE_VERSION@

 SUBDIRS = src
+prefix = @prefix@

 DISTFILES = COPYING CREDITS FeatureLog GNUmakefile.in README TODO \
        graphthing.1.in \
--- graphthing-1.3.1/src/GNUmakefile.in.orig	2006-09-19 13:57:21.100417848 +0000
+++ graphthing-1.3.1/src/GNUmakefile.in	2006-09-19 13:57:30.039058968 +0000
@@ -78,7 +78,7 @@
 
 lang.cc : phrases.h
 phrases.h : phrases.lang
-	@./init-phrases.sh
+	@./init-phrases.pl
 
 #%.o : %.c
 #	@echo "$< -> $@"

Attachment: pgpp76lkdnDrM.pgp
Description: PGP signature