Bug 339995

Summary:	app-text/htp-1.16: doesn't compile on amd64 (with possible fix)
Product:	Gentoo Linux	Reporter:	James Cline <james.cline>
Component:	Current packages	Assignee:	Gentoo Web Application Packages Maintainers <web-apps>
Status:	RESOLVED FIXED
Severity:	normal	CC:	sam, stefan
Priority:	High	Keywords:	PATCH
Version:	unspecified
Hardware:	AMD64
OS:	Linux
URL:	http://sourceforge.net/tracker/?func=detail&aid=3082574&group_id=41382&atid=430444
Whiteboard:
Package list:		Runtime testing required:	---
Attachments:	Ebuild with amd64 fix Patch to make compilation on amd64 possible

Description James Cline 2010-10-06 23:50:11 UTC

emerge app-text/htp fails on amd64.

Similarly, downloading the source and compiling it with make fails.




Reproducible: Always

Steps to Reproduce:
1. Add "app-text/htp **" to /etc/portage/package.keywords
2. emerge app-text/htp

Actual Results:  
It fails to build and install.

Expected Results:  
Built and installed.

Relevant bug I filed on sourceforge: http://sourceforge.net/tracker/?func=detail&aid=3082574&group_id=41382&atid=430444

Diff of potential fix, file src/snprintf.c
158c158
< #define VA_COPY(dest, src) (dest) = (src)
---
> #define VA_COPY(dest, src) va_copy(dest, src)

Comment 1 Stefan Wimmer 2010-12-18 14:18:45 UTC

Same here - did you resolve it already?

Comment 2 Stefan Wimmer 2010-12-18 16:55:25 UTC

(In reply to comment #1)
> Same here - did you resolve it already?

I apologize - I first should have read and tried the solution you proposed :-/
I made a new patch-file and modified the ebuild and everything installs just fine. I also tested the new ebuild with patch on a x86 system and it compiled and works as far as I can say ...

Just in case you want to try that as well I'll attach both the modified ebuild + patch.

Comment 3 Stefan Wimmer 2010-12-18 16:56:03 UTC

Created attachment 257500 [details]
Ebuild with amd64 fix

Comment 4 Stefan Wimmer 2010-12-18 16:56:56 UTC

Created attachment 257502 [details, diff]
Patch to make compilation on amd64 possible

Comment 5 Sam James archtester

2023-10-06 01:05:11 UTC

This one was solved a long time ago.