[GRASS5] Diffs for grass 5.0.2 package for debian woody now available.

[Glynn Clements]

giuseppe wrote:

> The diff against official grass-5.0.2_src.tar.gz are ready as promised. The 
> workstation is up and running and if things hold I hope to answer without 
> lag. The grass-5.0.2_src.tar.gz I used today to make the diffs is the same I 
> used when making  the source debian package. Today I downloaded:
> ftp://grass.itc.it/pub/grass/grass5/source/grass-5.0.2_src.tar.gz
> and made a cmp against my old grass-5.0.2_src.tar.gz to ensure they are the 
> same.
> 
> As usually, you can find everything at: 
> http://ezralabs.esaurito.net/repository/debian/grass/
> 
> In particular: 
> http://ezralabs.esaurito.net/repository/debian/grass/grass_5.0.2-1.diff.gz

Looking at the diff, I have a couple of questions:

1. "configure" has already been run, and lex/yacc source files have be
processed to produce the intermediate C files. Is this due to Debian
policy, or is it an artifact of the way that the package was produced?

2. What was the purpose of this patch:

--- grass-5.0.2.orig/src/display/devices/lib/Text2.c
+++ grass-5.0.2/src/display/devices/lib/Text2.c
@@ -170,9 +170,11 @@
 	double text_size_x,double text_size_y,double text_rotation,
 	char *string)
 {
-	double sinrot = sin (D2R (text_rotation)) ;
-	double cosrot = cos (D2R (text_rotation)) ;
-
+double sinrot,cosrot;
+	//double sinrot = sin (D2R (text_rotation)) ;
+	//double cosrot = cos (D2R (text_rotation)) ;
+	sinrot = sin (D2R (text_rotation)) ;
+	cosrot = cos (D2R (text_rotation)) ;
 	am_inside = 0 ;
 	curx = basex = (double)x ;
 	cury = basey = (double)y ;

Is this due to a gcc bug?

3. What is the purpose of debian/fixpaths.sh? The entire GRASS
installation (i.e. everything under $prefix/grass5) is supposed to be
relocatable. If the installation is moved, the only thing that should
need to be changed is the definition of GISBASE in the grass5 script.

-- 
Glynn Clements <glynn.clements at virgin.net>