Odd issue with pod2man when building on Fedora build server

Julien ÉLIE julien at trigofacile.com
Fri May 24 19:48:57 UTC 2013


Hi Jochen,

I have just had a look at the existing patches in your Fedora build.
Do you believe some of them would be worthwhile taking into account in upstream?



Here are a few comments:



--- inn-2.4.3/innfeed/imap_connection.c.warn	2006-03-20 05:14:57.000000000 +0100
+++ inn-2.4.3/innfeed/imap_connection.c	2006-03-27 12:07:20.000000000 +0200
@@ -2452,7 +2452,7 @@
     char *p;
 
 #ifdef HAVE_SASL
-    const char *mechusing;
+    const char *mechusing = NULL;
     int saslresult=SASL_NOMECH;
 
     sasl_interact_t *client_interact=NULL;


=> Is it really necessary?
Which warning is suppressed with this patch?  I do not see one when building.
We can have this change in upstream if it fixes something for Fedora.







--- inn-2.5.1/doc/pod/rc.news.pod.nologin	2009-10-12 20:24:04.000000000 +0200
+++ inn-2.5.1/doc/pod/rc.news.pod	2009-10-13 21:53:35.000000000 +0200
@@ -63,16 +63,16 @@ is recommended to throttle the server fi
 
 To start INN and leave certain error messages going to the terminal:
 
-       su - news -c <pathbin in inn.conf>/rc.news
+       su -m news -c /etc/rc.news


=> Is rc.news really installed as /etc/rc.news in Fedora, and not in a subdirectory
of /etc?
Maybe the "-m" option could be used in our examples; I will have a look.






--- inn-2.5.3/innfeed/configfile.l.lex	2012-09-11 14:29:00.473000388 +0200
+++ inn-2.5.3/innfeed/configfile.l	2012-09-11 14:29:40.108999869 +0200
@@ -36,7 +36,7 @@ static void yyunput(int, char *) UNUSED;
 int yyget_lineno(void);
 FILE *yyget_in(void);
 FILE *yyget_out(void);
-int yyget_leng(void);
+// int yyget_leng(void);
 char *yyget_text(void);
 int yyget_debug(void);
 void yyset_lineno(int);


=> What problem are fixing with this change?






--- inn-2.5.2/storage/cnfs/cnfs.h.posix	2010-03-24 21:10:36.000000000 +0100
+++ inn-2.5.2/storage/cnfs/cnfs.h	2010-06-28 17:33:11.106804964 +0200
@@ -18,4 +18,12 @@ void cnfs_printfiles(FILE *file, TOKEN t
 char *cnfs_explaintoken(const TOKEN token);
 void cnfs_shutdown(void);
 
+/* Patch for POSIX  */
+
+#ifndef MADV_SEQUENTAL
+  #ifdef POSIX_MADV_SEQENTIAL
+    #define MADV_SEQUITIAL POSIX_MADV_SEQUENTIAL
+  #endif
+#endif
+
 #endif


=> I will have a look at this one.  Is there a specific architecture that
needs this definition?






--- inn-2.5.1/storage/buffindexed/ovmethod.mk.shared	2009-12-01 16:50:37.174811272 +0100
+++ inn-2.5.1/storage/buffindexed/ovmethod.mk	2009-12-01 16:53:24.914809560 +0100
@@ -1,8 +1,8 @@
 # This rule requires a compiler that supports -o with -c.  Since it's normally
 # used by developers, that should be acceptable.
-buffindexed/buffindexed_d.o: buffindexed/buffindexed.c
-	$(CC) $(CFLAGS) -DBUFF_DEBUG -c -o $@ buffindexed/buffindexed.c
+buffindexed/buffindexed_d.$(EXTOBJ): buffindexed/buffindexed.c
+	$(LIBCC) $(CFLAGS) -DBUFF_DEBUG -c -o $@ buffindexed/buffindexed.c


=> Do you have a specific need for shared libraries?






--- inn-2.5.1/frontends/Makefile.pie	2009-10-12 20:24:04.000000000 +0200
+++ inn-2.5.1/frontends/Makefile	2009-10-13 21:51:38.000000000 +0200
@@ -85,7 +85,7 @@ ovdb_init:	ovdb_init.o    $(BOTH)   ; $(
 ovdb_monitor:	ovdb_monitor.o $(BOTH)   ; $(LINK) ovdb_monitor.o $(STORELIBS)
 ovdb_server:	ovdb_server.o  $(BOTH)   ; $(LINK) ovdb_server.o  $(STORELIBS)
 ovdb_stat:	ovdb_stat.o    $(BOTH)   ; $(LINK) ovdb_stat.o    $(STORELIBS)
-rnews:		rnews.o        $(LIBINN) ; $(LINK) rnews.o        $(STORELIBS)
+rnews:		rnews.o        $(LIBINN) ; $(LINK) rnews.o        -pie $(STORELIBS)
 sm:		sm.o           $(BOTH)   ; $(LINK) sm.o           $(STORELIBS)
 

=> Is it necessary to add "-pie" for rnews?






--- inn-2.4.1/Makefile.old	2004-12-13 13:09:57.000000000 +0100
+++ inn-2.4.1/Makefile	2004-12-13 14:29:56.625685696 +0100
@@ -91,9 +91,9 @@
 directories:
 	@chmod +x support/install-sh
 	for D in $(INSTDIRS) ; do \
-	    support/install-sh $(OWNER) -m 0755 -d $(D)$$D ; \
+	    support/install-sh $(OWNER) -m 0755 -d $(DESTDIR)/$$D ; \
 	done
-	support/install-sh $(OWNER) -m 0750 -d $(D)$(PATHRUN)
+	support/install-sh $(OWNER) -m 0750 -d $(DESTDIR)/$(PATHRUN)



=> Be careful...  Makefile.global mentions:

##  DESTDIR should have a trailing slash, as the trailing
##  slash is not added automatically (in case someone wants to add a prefix
##  that isn't just a parent directory).

DESTDIR         =
D               = $(DESTDIR)

so your patch normally should not add "/".  Anyway, as $(D) is $(DESTDIR),
the patch is not necessary.


-- 
Julien ÉLIE

« – Ne me dis pas que c'est cette poignée de Gaulois d'en face qui
    vous a mis dans cet état !
  – Il est vrai que j'ai été frappé moi-même par la force
    persuasive de cette poignée, amiral ! » (Astérix)


More information about the inn-workers mailing list