Skip site navigation (1)Skip section navigation (2)
Date:      Tue, 20 Jul 2010 15:19:44 +0200
From:      Horvath Andras <han@log69.com>
To:        Baptiste Daroussin <baptiste.daroussin@gmail.com>
Cc:        freebsd-ports-bugs@freebsd.org
Subject:   Re: ports/148773: [PATCH] graphics/aaphoto: update to 0.38
Message-ID:  <20100720151944.00afa45a@dell.andras>
In-Reply-To: <AANLkTimghNiEBjfOPxruMAIYAUD8XwwdN1MebuOGiwI0@mail.gmail.com>
References:  <201007200653.o6K6rMaM079521@freefall.freebsd.org> <20100720091710.36452d59@dell.andras> <20100720073407.GA11445@tuxaco.net> <20100720095756.11d1c74b@dell.andras> <20100720083801.GC11445@tuxaco.net> <20100720123202.5db1c268@dell.andras> <AANLkTimghNiEBjfOPxruMAIYAUD8XwwdN1MebuOGiwI0@mail.gmail.com>

next in thread | previous in thread | raw e-mail | index | archive | help
--MP_/3oTOfkl+STbYoCahuVkPxLb
Content-Type: text/plain; charset=ISO-8859-1
Content-Transfer-Encoding: quoted-printable
Content-Disposition: inline

I've tested the USE_AUTOTOOLS=3D autoheader:262 setting, and it seems
to be working good. Thanks for the suggestion.

I attached the new patch with it. Here's the log:
http://pastebin.org/408322

Andras


On Tue, 20 Jul 2010 11:43:01 +0000
Baptiste Daroussin <baptiste.daroussin@gmail.com> wrote:

> Le 20 juillet 2010 10:32, Horvath Andras <han@log69.com> a =E9crit :
> > Thanks for the test.
> >
> > I think the patch should be good then. Will you commit it if noone
> > else has problem with it?
> >
>=20
> There is still a problem with the autotools  :
> /usr/ports/graphics/aaphoto_dsx/work/aaphoto-0.38/missing:
> autoheader: not found WARNING: `autoheader' is missing on your
> system.  You should only need it if you modified `acconfig.h' or
> `configure.ac'.  You might want to install the `Autoconf' and `GNU
> m4' packages.  Grab them from any GNU archive site.
>=20
> We can live with that but it would be better to fix it.
>=20
> I think adding
> USE_AUTOTOOLS=3D autoheader:262 (I don't know if it is the good version)
> should fix it, I don't have time to test currently, I'll do later if
> noone has done it since :)

--MP_/3oTOfkl+STbYoCahuVkPxLb
Content-Type: text/plain
Content-Transfer-Encoding: 7bit
Content-Disposition: attachment; filename=patch.txt

diff -ru graphics/aaphoto.old/Makefile graphics/aaphoto/Makefile
--- graphics/aaphoto.old/Makefile	2010-06-03 09:56:16.000000000 +0200
+++ graphics/aaphoto/Makefile	2010-07-20 15:02:10.000000000 +0200
@@ -6,8 +6,7 @@
 #
 
 PORTNAME=	aaphoto
-PORTVERSION=	0.37
-PORTREVISION=	1
+PORTVERSION=	0.38
 CATEGORIES=	graphics
 MASTER_SITES=	http://log69.com/downloads/
 DISTNAME=	${PORTNAME}_sources_v${PORTVERSION}
@@ -15,10 +14,13 @@
 MAINTAINER=	han@log69.com
 COMMENT=	Auto Adjust Photo, automatic color correction of photos
 
+LICENSE=       GPLv3
+
 LIB_DEPENDS=	jasper:${PORTSDIR}/graphics/jasper \
 		jpeg:${PORTSDIR}/graphics/jpeg \
 		png:${PORTSDIR}/graphics/png
 
+USE_AUTOTOOLS=	autoheader:262
 GNU_CONFIGURE=	yes
 CONFIGURE_ENV=	CPPFLAGS="-I${LOCALBASE}/include" \
 		LDFLAGS="-L${LOCALBASE}/lib"
@@ -31,11 +33,7 @@
 
 .include <bsd.port.pre.mk>
 
-.if ${ARCH} != "amd64" && ${ARCH} != "i386"
-BROKEN=		Does not compile on !amd64 and !i386
-.endif
-
-.if ${OSVERSION} >= 700000
+.if ${ARCH} == "amd64" || ${ARCH} == "i386" && ${OSVERSION} >= 700000
 CFLAGS+=	-fopenmp -D__OPENMP__
 .endif
 
diff -ru graphics/aaphoto.old/distinfo graphics/aaphoto/distinfo
--- graphics/aaphoto.old/distinfo	2010-05-10 15:16:39.000000000 +0200
+++ graphics/aaphoto/distinfo	2010-07-20 15:00:11.000000000 +0200
@@ -1,3 +1,3 @@
-MD5 (aaphoto_sources_v0.37.tar.gz) = f9cfaf7112713cb937f7b3ac1236d077
-SHA256 (aaphoto_sources_v0.37.tar.gz) = 2efe35c406ea5bd0b8a05a6eab02437c6b3eefbff38d4d74c0f200023c4f7ae4
-SIZE (aaphoto_sources_v0.37.tar.gz) = 134442
+MD5 (aaphoto_sources_v0.38.tar.gz) = 99c07a650eda444e753cb611dd0c9d36
+SHA256 (aaphoto_sources_v0.38.tar.gz) = e2a4de457d957bb80925beb18c79b4128982941dec24f506dfa2d495830ce63e
+SIZE (aaphoto_sources_v0.38.tar.gz) = 134661
diff -ru graphics/aaphoto.old/files/patch-Makefile.in graphics/aaphoto/files/patch-Makefile.in
--- graphics/aaphoto.old/files/patch-Makefile.in	2010-05-23 15:21:24.000000000 +0200
+++ graphics/aaphoto/files/patch-Makefile.in	2010-07-20 14:59:05.000000000 +0200
@@ -5,7 +5,7 @@
  top_builddir = @top_builddir@
  top_srcdir = @top_srcdir@
 -AM_CFLAGS = -Wall -O2 -D__OPENMP__ -fopenmp
-+AM_CFLAGS = -Wall -O2 -Wno-unknown-pragmas
++AM_CFLAGS = -Wall -O2
  AM_LDFLAGS = -ljasper -ljpeg -lpng -lm
  aaphoto_SOURCES = aaphoto.c
  INCLUDES = 
Only in graphics/aaphoto: work

--MP_/3oTOfkl+STbYoCahuVkPxLb--



Want to link to this message? Use this URL: <https://mail-archive.FreeBSD.org/cgi/mid.cgi?20100720151944.00afa45a>