Bug#812682: w3m FTCBFS: uses build architecture compiler

[Manuel A. Fernandez Montecelo]

2017-12-12 15:46 GMT+01:00 Tatsuya Kinoshita :
> On December 11, 2017 at 10:36PM +0100, manuel.montezelo (at gmail.com) wrote:
>>>Tatsuya, what do you think about applying this fix?
>>>
>> I just uploaded an NMU with the patch applied, to delayed/15.  I
>> pushed
>> the changes to the repository.
>>
>> If you disagree, please tell me and I can cancel the NMU, and can
>> revert
>> the changes as well; or you can do it yourself if you prefer.
>>
>> If it's OK for you, I can re-schedule it to happen sooner.
>
> Agreed.
>
> Sorry for the late reply.

Re-scheduled, thanks!


-- 
Manuel A. Fernandez Montecelo 

Bug#812682: w3m FTCBFS: uses build architecture compiler

[Tatsuya Kinoshita]

On December 11, 2017 at 10:36PM +0100, manuel.montezelo (at gmail.com) wrote:
>>Tatsuya, what do you think about applying this fix?
>>
> I just uploaded an NMU with the patch applied, to delayed/15.  I
> pushed
> the changes to the repository.
>
> If you disagree, please tell me and I can cancel the NMU, and can
> revert
> the changes as well; or you can do it yourself if you prefer.
>
> If it's OK for you, I can re-schedule it to happen sooner.

Agreed.

Sorry for the late reply.

Thanks,
--
Tatsuya Kinoshita


pgpDEPh8AOmJT.pgp
Description: PGP signature

Bug#812682: w3m FTCBFS: uses build architecture compiler

[Manuel A. Fernandez Montecelo]

Hi,

2017-10-31 20:43 Manuel A. Fernandez Montecelo:

Hi

2016-01-25 22:02 Helmut Grohne:

Source: w3m
Version: 0.5.3-26
User: heml...@debian.org


Helmut: wrong user...



Usertags: rebootstrap

Hi,

w3m fails to cross build from source for a number of reasons. Most
immediately, configure is run without --host and thus configures for the
build architecture. Even though configure has code for detecting the
name of pkg-config, it still falls back to calling it directly causing
it to miss libraries such as imlib2. I am attaching a patch for these
issues. Even after applying it, the build fails due to executing
./mknames. Since mknames is executed during build and not installed into
any package, it should be compiled with the build architecture compiler.
For this to work, it needs to stop #including things such as config.h
however. Also object files such as Str.o need to be built for both
architectures. I haven't figured out the details yet, so my patch is
only a partial solution. Hope it helps whomever is looking into cross
compiling w3m at a later time.


Tatsuya, what do you think about applying this fix?

Lots of packages build-depent on w3m (x11proto* and basic X libraries,
mutt...), it would be nice to be able to cross-build it from other
systems, for bootstrapping new architectures and similar scenarios.

Cheers.


I just uploaded an NMU with the patch applied, to delayed/15.  I pushed
the changes to the repository.

If you disagree, please tell me and I can cancel the NMU, and can revert
the changes as well; or you can do it yourself if you prefer.

If it's OK for you, I can re-schedule it to happen sooner.

Let's hope that this push helps to move things forward to make possible
to cross-compile this important package :)


Cheers and thanks for maintaining this package!

--
Manuel A. Fernandez Montecelo 

Bug#812682: w3m FTCBFS: uses build architecture compiler

[Manuel A. Fernandez Montecelo]

Hi

2016-01-25 22:02 Helmut Grohne:

Source: w3m
Version: 0.5.3-26
User: heml...@debian.org


Helmut: wrong user...



Usertags: rebootstrap

Hi,

w3m fails to cross build from source for a number of reasons. Most
immediately, configure is run without --host and thus configures for the
build architecture. Even though configure has code for detecting the
name of pkg-config, it still falls back to calling it directly causing
it to miss libraries such as imlib2. I am attaching a patch for these
issues. Even after applying it, the build fails due to executing
./mknames. Since mknames is executed during build and not installed into
any package, it should be compiled with the build architecture compiler.
For this to work, it needs to stop #including things such as config.h
however. Also object files such as Str.o need to be built for both
architectures. I haven't figured out the details yet, so my patch is
only a partial solution. Hope it helps whomever is looking into cross
compiling w3m at a later time.


Tatsuya, what do you think about applying this fix?

Lots of packages build-depent on w3m (x11proto* and basic X libraries,
mutt...), it would be nice to be able to cross-build it from other
systems, for bootstrapping new architectures and similar scenarios.

Cheers.

--
Manuel A. Fernandez Montecelo 

Bug#812682: w3m FTCBFS: uses build architecture compiler

[Helmut Grohne]

Source: w3m
Version: 0.5.3-26
User: heml...@debian.org
Usertags: rebootstrap

Hi,

w3m fails to cross build from source for a number of reasons. Most
immediately, configure is run without --host and thus configures for the
build architecture. Even though configure has code for detecting the
name of pkg-config, it still falls back to calling it directly causing
it to miss libraries such as imlib2. I am attaching a patch for these
issues. Even after applying it, the build fails due to executing
./mknames. Since mknames is executed during build and not installed into
any package, it should be compiled with the build architecture compiler.
For this to work, it needs to stop #including things such as config.h
however. Also object files such as Str.o need to be built for both
architectures. I haven't figured out the details yet, so my patch is
only a partial solution. Hope it helps whomever is looking into cross
compiling w3m at a later time.

Helmut
diff --minimal -Nru w3m-0.5.3/debian/changelog w3m-0.5.3/debian/changelog
--- w3m-0.5.3/debian/changelog  2015-12-01 16:36:41.0 +0100
+++ w3m-0.5.3/debian/changelog  2016-01-25 21:14:20.0 +0100
@@ -1,3 +1,12 @@
+w3m (0.5.3-26.1) UNRELEASED; urgency=medium
+
+  * Non-maintainer upload.
+  * Partially fix FTCBFS (addresses: #-1)
++ Pass --host to configure
++ export PKG_CONFIG
+
+ -- Helmut Grohne   Mon, 25 Jan 2016 20:57:02 +0100
+
 w3m (0.5.3-26) unstable; urgency=medium
 
   * Update 020_debian.patch to v0.5.3+git20151119
diff --minimal -Nru w3m-0.5.3/debian/rules w3m-0.5.3/debian/rules
--- w3m-0.5.3/debian/rules  2015-11-22 19:08:46.0 +0100
+++ w3m-0.5.3/debian/rules  2016-01-25 21:15:24.0 +0100
@@ -1,5 +1,7 @@
 #!/usr/bin/make -f
 
+include /usr/share/dpkg/architecture.mk
+
 #export DH_VERBOSE=1
 
 export DEB_BUILD_MAINT_OPTIONS = hardening=+all
@@ -28,6 +30,11 @@
--enable-image=$(DEVS) \
--enable-m17n --enable-unicode --enable-nls
 
+ifneq ($(DEB_BUILD_GNU_TYPE),$(DEB_HOST_GNU_TYPE))
+confargs += --build=$(DEB_BUILD_GNU_TYPE) --host=$(DEB_HOST_GNU_TYPE)
+export PKG_CONFIG=$(DEB_HOST_GNU_TYPE)-pkg-config
+endif
+
 builddir := .
 
 build-indep: