Bug 125260

Summary: archivers/pbzip2: PRIu64 macro not being expanded correctly and leaking into -v output
Product: Ports & Packages Reporter: Geoff The <geoff2k>
Component: Individual Port(s)Assignee: Dmitry Marakasov <amdmi3>
Status: Closed FIXED    
Severity: Affects Only Me    
Priority: Normal    
Version: Latest   
Hardware: Any   
OS: Any   

Description Geoff The 2008-07-04 12:50:02 UTC 
The PRIu64 macros being used to output byte sizes are not being expanded correctly.  E.g.

% /usr/local/bin/pbzip2 -rv AUTHORS 
Parallel BZIP2 v1.0.2 - by: Jeff Gilchrist [http://compression.ca]
[July 25, 2007]             (uses libbzip2 by Julian Seward)

         # CPUs: 2
 BWT Block Size: 900k
File Block Size: 900k
-------------------------------------------
         File #: 1 of 1
     Input Name: AUTHORS
    Output Name: AUTHORS.bz2

     Input Size: PRIu64 bytes
Compressing data...
    Output Size: PRIu64 bytes
-------------------------------------------

     Wall Clock: 0.052543 seconds

Fix: 

Locally, I've replaced all instances of PRIu64 with llu, but this is platform-specific.
Comment 1 Edwin Groothuis freebsd_committer freebsd_triage 2008-07-04 12:50:16 UTC 
Responsible Changed
From-To: freebsd-ports-bugs->infofarmer

Over to maintainer (via the GNATS Auto Assign Tool)
Comment 2 Pav Lucistnik freebsd_committer freebsd_triage 2008-09-05 17:12:00 UTC 
Hi folks, the problem is caused by incorrect patching in the port.
The fix is attached. According to cvs history, the patching was made to
allow compilation on FreeBSD 4.x, which we don't support anymore, so it
can be just removed.

Andrew, do you approve this patch?

Index: Makefile
===================================================================
RCS file: /home/pcvs/ports/archivers/pbzip2/Makefile,v
retrieving revision 1.14
diff -a -u -r1.14 Makefile
--- Makefile	30 Jul 2007 16:24:27 -0000	1.14
+++ Makefile	5 Sep 2008 16:09:05 -0000
@@ -9,9 +9,10 @@
 PORTNAME=	pbzip2
 PORTVERSION=	1.0.2.0.2
 DISTVERSION=	1.0.2
+PORTREVISION=	1
 CATEGORIES=	archivers
 MASTER_SITES=	http://compression.ca/pbzip2/ CENKES
-DIST_SUBDIR=	${PKGNAME}
+DIST_SUBDIR=	${PORTNAME}-${PORTVERSION}
 
 MAINTAINER=	infofarmer@FreeBSD.org
 COMMENT=	Parallel BZIP2
@@ -27,8 +28,6 @@
 		s,-O3,${CXXFLAGS},; \
 		s,^CC,#CC,; \
 		s,CC,CXX,' ${WRKSRC}/Makefile
-	@${REINPLACE_CMD} -e 's,PRIu64 "Lu",PRIu64 Lu,;s,"PRIu64",#PRIu64,g' \
-		${WRKSRC}/pbzip2.cpp
 
 do-install:
 	@${INSTALL_PROGRAM} ${WRKSRC}/${PORTNAME} ${PREFIX}/bin/

-- 
Pav Lucistnik <pav@oook.cz>
              <pav@FreeBSD.org>

The Linimon's Rule: The More You Close, The More Will Come
Comment 3 Mark Linimon freebsd_committer freebsd_triage 2008-09-07 01:21:40 UTC 
Responsible Changed
From-To: infofarmer->freebsd-ports-bugs

Maintainer was reset.
Comment 4 Dmitry Marakasov freebsd_committer freebsd_triage 2008-09-12 15:22:25 UTC 
Responsible Changed
From-To: freebsd-ports-bugs->amdmi3

I'll take it.
Comment 5 dfilter service freebsd_committer freebsd_triage 2008-09-23 14:52:32 UTC 
amdmi3      2008-09-23 13:52:28 UTC

  FreeBSD ports repository

  Modified files:
    archivers/pbzip2     Makefile 
  Log:
  - Remove bogus REINPLACE_CMD which was intended to fix build on 4.x, but has broken program output instead.
  - While here, line up MAN and MLINKS lines
  
  PR:             125260
  Submitted by:   Geoff The <geoff2k at gmail dot com>
  Patch by:       pav@
  
  Revision  Changes    Path
  1.16      +4 -5      ports/archivers/pbzip2/Makefile
_______________________________________________
cvs-all@freebsd.org mailing list
http://lists.freebsd.org/mailman/listinfo/cvs-all
To unsubscribe, send any mail to "cvs-all-unsubscribe@freebsd.org"
Comment 6 Dmitry Marakasov freebsd_committer freebsd_triage 2008-09-23 14:52:38 UTC 
State Changed
From-To: open->closed

Committed. Thanks!