savannah: Build system does not allow cross compiling

[slavazanko]

Original: ​http://savannah.gnu.org/bugs/?16383

Submitted by:	Rolf Offermanns <roffermanns>	Submitted on:	Wed 19 Apr 2006 01:23:19 PM UTC
Category:	None	Severity:	3 - Normal
Status:	None	Privacy:	Public
Assigned to:	None	Open/Closed:	Open
Release:	4.6.1	Operating System:	GNU/Linux

Discussion:

Thu 22 Jun 2006 06:56:43 AM UTC, comment #3:
The major problem here is that man2hlp relies on glib. This 
complicates the task of making it build in cross environment. One 
way to fix this is to drop the glib dependency - there are tree or 
four functions which need to be replaced. When I have time I'll do 
this if noone beats me to it. I have already made some changes to 
the auto stuff so that MC will build properly in cross environment
but whithout changing man2hlp they are of no use.
	Pavel Tsekov <ptsekov>
Project Administrator
Thu 22 Jun 2006 06:16:38 AM UTC, comment #2:
I would, but I have no clue about the auto* stuff. Sorry.
	Rolf Offermanns <roffermanns>
Wed 21 Jun 2006 05:37:31 PM UTC, comment #1:
Please provide a patch. Thanks.
	Leonard den Ottolander <leonardjo>
Project Member
Wed 19 Apr 2006 01:23:19 PM UTC, original submission:
The man2hlp program is build using the target compiler. The 
resulting executable therefore does not work on the build host. 
Since the man2hlp program is needed during compile time, the build 
will fail.
The auto* stuff must be changed to include a CC_FOR_BUILD macro 
which must be used for the man2hlp (and any other used-at-
compile-time) application. 

[metux]

I really vote for getting rid of glib.
Instead we should write some tiny lib with the few primary things needed from glib.

[metux]

IMHO we just should rewrite the whole man2hlp program in perl.

Maybe takes a bit more work in the first place, but should save a lot hassle in the future.

[slavazanko]

Created branch 45_rewrite_man2hlp
Initial changeset:649487fac17bfdd0bc9ef39c6e0852802669b1e6

Review, please.

[angel_il]

thanx for Alexander Prenko for good work

[slavazanko]

merge changeset:47495258205a66eb773abe8f2460b073f62bf4ca

set of commits visible by command:

git log --pretty=oneline 649487fac17bfdd0bc9ef39c6e0852802669b1e6^..878a0fdc85e063f594fa622bb08717eece2a48e6

[angel_il]

diff --git a/src/man2hlp/man2hlp.in b/src/man2hlp/man2hlp.in
index 18d9226..acfb078 100644
--- a/src/man2hlp/man2hlp.in
+++ b/src/man2hlp/man2hlp.in
@@ -413,7 +413,7 @@ sub handle_alt_font($)
}

# Skip more spaces
- shift @p while $p[0] eq ' ';
+ shift @p while @p && $p[0] eq ' ';

next;
}

[angel_il]

branch: 45_man2hlp_warnings_fix
changeset: eafa23a2a753f60231bd996fe00bbbb375f0b9b6

[angel_il]

Fixed: 385d5bea4181b1a411d6896d1daa394c7ba78856

[zaytsev]

It's a stable candidate, sorry for closing prematurely.

[angel_il]

branch: 45_LANG_warnings_fix
changeset: 5c6154538110c45bcf0357535e3aaf0d2353aa68

this is branch need for fix warnings like this:

perl: warning: Setting locale failed.
perl: warning: Please check that your locale settings:
        LANGUAGE = (unset),
        LC_ALL = (unset),
        LC_COLLATE = "C",
        LANG = "pl"
    are supported and installed on your system.
perl: warning: Falling back to the standard locale ("C").

[zaytsev]

I already committed a similar patch in the man branch. Also, I discovered that LANG is not needed at all, it's just a coincidence, so if you definitely want to use your branch instead of mine, remove them altogether.

[andrew_b]

Fixed in #2243.
changeset:06e3dbbae9189197d0b5fa0687deba3b001935da