Bug 12511 – New ebuild for "wdiff" progrom

• Status: RESOLVED FIXED
• Product: Gentoo Linux

[Description Peter Simons 2002-12-21 03:27:14 UTC]

I hope this is the right place to submit a new ebuild? Appended below is one for
the wdiff package. I guess, this one should go to 'app-text':

# Copyright 1999-2002 Gentoo Technologies, Inc.
# Distributed under the terms of the GNU General Public License v2
# $Header$

IUSE=""

S=${WORKDIR}/${P}
DESCRIPTION="Create a diff disregarding formatting"
SRC_URI="ftp://ftp.gnu.org/gnu/wdiff/${P}.tar.gz"
HOMEPAGE="http://www.gnu.org/software/diffutils/diffutils.html"

KEYWORDS="~x86"
SLOT="0"
LICENSE="GPL-2"

RDEPEND="virtual/glibc"

DEPEND="${RDEPEND}
	sys-libs/libtermcap-compat"

src_compile() {
        # Cannot use econf here because the configure script that
        # comes with wdiff is too old to understand the standard
        # options.

	./configure --prefix=/usr \
	    || die
	emake || die
}

src_install() {
	make prefix=${D}/usr \
		datadir=${D}/usr/share \
		infodir=${D}/usr/share/info \
		mandir=${D}/usr/share/man \
		install || die

	if [ -z "`use build`" ]
	then
		dodoc COPYING ChangeLog NEWS README
	else
		rm -rf ${D}/usr/share/info
	fi
}

[Comment 1 Peter Simons 2002-12-21 03:29:32 UTC]

I guess I was too quick, sorry. :-( The wdiff package depends on

   sys-apps/diffutils

as well, because it is just a mere front-end to the normal diff utility.

[Comment 2 Seemant Kulleen (RETIRED) 2002-12-21 03:34:56 UTC]

Peter, where the heck are you?  Respond to my emails, dammit! :)

[Comment 3 Peter Simons 2002-12-21 03:42:02 UTC]

As ordered, Sir. :-) Replies are underway ...

[Comment 4 Seemant Kulleen (RETIRED) 2002-12-21 06:04:01 UTC]

gcc -c -DHAVE_CONFIG_H -I. -I.  -march=i686 -O3 -fomit-frame-pointer -pipe wdiff.c
gcc -c -DHAVE_CONFIG_H -I. -I.  -march=i686 -O3 -fomit-frame-pointer -pipe
readpipe.c
wdiff.c:62: macro `strstr' used without args
wdiff.c:62: parse error before `__extension__'
gcc -c -DHAVE_CONFIG_H -I. -I.  -march=i686 -O3 -fomit-frame-pointer -pipe
writepipe.c
gcc -c -DHAVE_CONFIG_H -I. -I.  -march=i686 -O3 -fomit-frame-pointer -pipe error.c
gcc -c -DHAVE_CONFIG_H -I. -I.  -march=i686 -O3 -fomit-frame-pointer -pipe getopt.c
gcc -c -DHAVE_CONFIG_H -I. -I.  -march=i686 -O3 -fomit-frame-pointer -pipe getopt1.c
rm -f libwd.a
ar cru libwd.a error.o getopt.o getopt1.o  
ranlib libwd.a
wdiff.c: In function `main':
wdiff.c:1140: warning: return type of `main' is not `int'
make: *** [wdiff.o] Error 1

!!! ERROR: app-misc/wdiff-0.5 failed.
!!! Function src_compile, Line 26, Exitcode 2
!!! (no error message)


Any ideas, peter?

[Comment 5 Peter Simons 2002-12-28 06:58:33 UTC]

Created attachment 6801 [details]
New version of the ebuild, which fixes the compile error experienced by Seemant

The problem you had comes from a non-ANSI-conformant declaration of the main()
function in the source. I added an appropriate patch to the ebuild. The
appended tar.gz archive contains the whole thing now, digest, ebuild file, and
diff.

[Comment 6 Peter Simons 2003-01-28 18:47:02 UTC]

Any chance of seeing this in the portage tree any time soon? Just wondering, the
build has been stable here for a while now and I didn't hear anything yet.

[Comment 7 Seemant Kulleen (RETIRED) 2003-02-12 02:14:57 UTC]

yes sir, coming to portage in a few minutes

[Comment 8 Seemant Kulleen (RETIRED) 2003-02-12 03:03:05 UTC]

sorry for the delay, peter, added to portage.