Summary: | net-misc/htun-0.9.5 ~amd64 keyword request | ||
---|---|---|---|
Product: | Gentoo Linux | Reporter: | Andrew Savchenko <bircoph> |
Component: | [OLD] Keywording and Stabilization | Assignee: | No maintainer - Look at https://wiki.gentoo.org/wiki/Project:Proxy_Maintainers if you want to take care of it <maintainer-needed> |
Status: | RESOLVED FIXED | ||
Severity: | normal | CC: | bircoph |
Priority: | Normal | Keywords: | EBUILD, KEYWORDREQ, PATCH |
Version: | unspecified | ||
Hardware: | AMD64 | ||
OS: | Linux | ||
Whiteboard: | |||
Package list: | Runtime testing required: | --- | |
Bug Depends on: | |||
Bug Blocks: | 243502 | ||
Attachments: |
htun-0.9.6.ebuild
htun-0.9.6-glibc.patch htun-0.9.6-makefile.patch htun-0.9.6.ebuild |
Description
Andrew Savchenko
2013-03-07 08:34:43 UTC
Some issues detected: 1. Invalid Gentoo Copyright on line: 1 Should be: # Copyright 1999-2013 Gentoo Foundation 2. Are you using EAPI=0? 3. ebuild calls gcc directly, and uses -g flag (Added blocker to this bug) gcc -I../include -I. -g -D_REENTRANT -c y.tab.c 4. Missing || die at lines 21 and 30 5. The package does not compile in my box: gcc -I../include -I. -g -D_REENTRANT -c y.tab.c In file included from gram.y:31:0: ../include/common.h:139:15: warning: built-in function 'log' declared as non-function [enabled by default] gram.y:305:6: warning: conflicting types for 'die_error' [enabled by default] gram.y:205:21: note: previous implicit declaration of 'die_error' was here gram.y:322:6: warning: conflicting types for 'yyerror' [enabled by default] y.tab.c:2035:7: note: previous implicit declaration of 'yyerror' was here gram.y:327:6: warning: conflicting types for 'yy_error' [enabled by default] gram.y:199:21: note: previous implicit declaration of 'yy_error' was here gram.y:332:13: warning: conflicting types for 'set_ip' [enabled by default] gram.y:332:13: error: static declaration of 'set_ip' follows non-static declaration gram.y:154:17: note: previous implicit declaration of 'set_ip' was here gram.y:346:12: error: static declaration of 'get_answer' follows non-static declaration gram.y:122:56: note: previous implicit declaration of 'get_answer' was here gram.y:356:15: error: static declaration of 'set_base64_user_pass' follows non-static declaration gram.y:212:47: note: previous implicit declaration of 'set_base64_user_pass' was here make: *** [y.tab.o] Error 1 emake failed The ebuild really needs some work. The package installs fine for me though. Also, there seems to be a 0.9.6 version out there. I've updated the package: * fixes copyright * updated ebuild (eapi=5) * add missing die's * fixes calling gcc directly * fixes dosen't respect users cflags/ldflags * version bump to 0.9.6 * added ~amd64 keyword * updated homepage (original seems to be offline, i took the hp from archs package) * updated glibc patch (a few things got fixed with the new version) The package builds fine for me too (on amd64). Please take a look at the ebuild. Hope it's mostly correct :) Created attachment 341340 [details]
htun-0.9.6.ebuild
ebuild for the latest version
Created attachment 341342 [details, diff]
htun-0.9.6-glibc.patch
updated glibc patch
Created attachment 341344 [details, diff]
htun-0.9.6-makefile.patch
makefile patch
(In reply to comment #3) > I've updated the package: > * fixes copyright > * updated ebuild (eapi=5) > * add missing die's > * fixes calling gcc directly > * fixes dosen't respect users cflags/ldflags > * version bump to 0.9.6 > * added ~amd64 keyword > * updated homepage (original seems to be offline, i took the hp from archs > package) > * updated glibc patch (a few things got fixed with the new version) > > The package builds fine for me too (on amd64). Please take a look at the > ebuild. Hope it's mostly correct :) Thanks, works fine here on both ~x86 and ~amd64. Though repoman is not happy with patching in src_unpack(), this is fixed below. Created attachment 341428 [details]
htun-0.9.6.ebuild
Updated ebuild with epatch commands placed into src_prepare() as EAPI>=2 requires.
Nice =) Do you get these QA issues? * QA Notice: Package triggers severe warnings which indicate that it * may exhibit random runtime failures. * queue.c:155:5: warning: null argument where non-null required (argument 1) [-Wnonnull] * queue.c:223:5: warning: null argument where non-null required (argument 1) [-Wnonnull] * queue.c:263:5: warning: null argument where non-null required (argument 1) [-Wnonnull] Yes, I got them. At the first glance at the code, non-null requirements are outdated. + 12 Mar 2013; Sergey Popov <pinkbyte@gentoo.org> +htun-0.9.6.ebuild, + +files/htun-0.9.6-glibc.patch, +files/htun-0.9.6-makefile.patch, + +files/README.gentoo: + Version bump, wrt bug #460656. EAPI 5, add ~amd64 keyword, add epatch_user, + use readme.gentoo eclass instead of simple message in pkg_postinst, respect + CFLAGS, LDFLAGS and compiler. Thanks to Michael Mair-Keimberger and Andrew + Savchenko for ebuild improvements |