[Pkg-e-devel] [git/pkg-e] Building e17 optimized for i686-pc-linux-gnu

[Albin Tonnerre]

Hi,

There are a couple issues with this patch. First, you don't need to include
buildvars.mk, as it is already included by autotools.mk. And more importantly,
setting DEB_HOST_ARCH and DEB_BUILD_ARCH does not change anything regarding
optimization, as it does not change (nor do you change them manually) CFLAGS. So
in the end, the package is still compiled with -g -Wall -O2.
Moreover, e17 doesn't make much usage of the CPU, so you're not likely going to
see any benefit from compiling it using any special -march or -mcpu flag.

Cheers,
Albin

On Mon, May 05, 2008 at 02:32:51PM +0200, Sedat Dilek wrote :
> Hi,
> 
> I made a new improved version of debian/rules now with checking
> DEB_HOST_ARCH and DEB_BUILD_ARCH before setting to i686-linux-gnu.
> Falko will also import it in his repository, e.g emodules.
> 
> Can you please test this one on non-i386-arch, e.g. x86-64?
> 
> Regards,
> Sedat
> 
> P.S.: New file attached.
> 
> ----- SNIP -----
> 
> #!/usr/bin/make -f
> 
> include /usr/share/cdbs/1/class/autotools.mk
> include /usr/share/cdbs/1/rules/debhelper.mk
> include /usr/share/cdbs/1/rules/simple-patchsys.mk
> include /usr/share/cdbs/1/rules/buildvars.mk
> 
> SHELL := sh -e
> 
> ifeq (,$(DEB_BUILD_ARCH))
> 	DEB_HOST_ARCH  := $(shell dpkg-architecture -qDEB_HOST_ARCH)
> 	DEB_BUILD_ARCH := $(shell dpkg-architecture -qDEB_BUILD_ARCH)
> endif
> 
> ifeq ($(DEB_BUILD_ARCH),i386)
> 	DEB_HOST_GNU_TYPE  := $(shell export DEB_HOST_GNU_TYPE=i686-linux-gnu)
> 	DEB_BUILD_GNU_TYPE := $(shell export DEB_BUILD_GNU_TYPE=i686-linux-gnu)
> endif
> 
> install/e17-data::
> 	gzip -9 debian/tmp/usr/share/enlightenment/doc/*.txt
> 	rm debian/tmp/usr/share/enlightenment/COPYING
> 	rm debian/tmp/usr/share/enlightenment/data/fonts/*
> 
> install/e17::
> 	rm -r debian/tmp/usr/lib/enlightenment/modules/layout
> 	rm -r debian/tmp/usr/lib/enlightenment/modules/wizard
> 
> binary-predeb/e17::
> 	chmod 4755 debian/e17/usr/lib/enlightenment/enlightenment_sys
> 	chmod 4755 debian/e17/usr/lib/enlightenment/modules/cpufreq/*/freqset
> 	chmod 4755 debian/e17/usr/lib/enlightenment/modules/temperature/*/tempget
> 
> DEB_INSTALL_MANPAGES_e17 := $(DEB_SRCDIR)/debian/enlightenment.1
> $(DEB_SRCDIR)/debian/enlightenment_*.1
> DEB_INSTALL_MANPAGES_e17-dev := $(DEB_SRCDIR)/debian/enlightenment-config.1
> DEB_DH_STRIP_ARGS := --dbg-package=e17-dbg
> DEB_CONFIGURE_EXTRA_FLAGS := --disable-rpath
> 
> ----- SNAP -----


> _______________________________________________
> Pkg-e-devel mailing list
> Pkg-e-devel at lists.alioth.debian.org
> http://lists.alioth.debian.org/mailman/listinfo/pkg-e-devel

-- 
Albin Tonnerre
-------------- next part --------------
A non-text attachment was scrubbed...
Name: not available
Type: application/pgp-signature
Size: 189 bytes
Desc: Digital signature
Url : http://lists.alioth.debian.org/pipermail/pkg-e-devel/attachments/20080505/afcc07d2/attachment.pgp