Ticket #45 (closed defect: fixed)
savannah: Build system does not allow cross compiling
Reported by: | roffermanns | Owned by: | slavazanko |
---|---|---|---|
Priority: | major | Milestone: | 4.7.3 |
Component: | mc-core | Version: | master |
Keywords: | Cc: | zaytsev | |
Blocked By: | Blocking: | ||
Branch state: | merged | Votes for changeset: | committed-master committed-stable |
Description (last modified by ossi) (diff)
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 |
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.
Comment 1 by Leonard den Ottolander <leonardjo> at Wed 21 Jun 2006 05:37:31 PM UTC:
Please provide a patch. Thanks.
Comment 2 by Rolf Offermanns <roffermanns> at Thu 22 Jun 2006 06:16:38 AM UTC:
I would, but I have no clue about the auto* stuff. Sorry.
Comment 3 by Pavel Tsekov <ptsekov> at Thu 22 Jun 2006 06:56:43 AM UTC:
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.
Change History
comment:2 Changed 16 years ago by 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.
comment:4 Changed 14 years ago by slavazanko
- Owner set to slavazanko
- Status changed from new to accepted
- severity set to on review
- Keywords stable-candidate added
- Milestone changed from Future Releases to 4.7.3
Created branch 45_rewrite_man2hlp
Initial changeset:649487fac17bfdd0bc9ef39c6e0852802669b1e6
Review, please.
comment:6 Changed 14 years ago by angel_il
- Votes for changeset changed from andrew_b to andrew_b angel_il
thanx for Alexander Prenko for good work
comment:7 Changed 14 years ago by slavazanko
- Status changed from accepted to testing
- Votes for changeset changed from andrew_b angel_il to committed-master
- Resolution set to fixed
- severity changed from on review to merged
merge changeset:47495258205a66eb773abe8f2460b073f62bf4ca
set of commits visible by command:
git log --pretty=oneline 649487fac17bfdd0bc9ef39c6e0852802669b1e6^..878a0fdc85e063f594fa622bb08717eece2a48e6
comment:8 Changed 14 years ago by 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; }
comment:9 Changed 14 years ago by angel_il
- Votes for changeset changed from committed-master to angel_il
- severity changed from merged to on review
branch: 45_man2hlp_warnings_fix
changeset: eafa23a2a753f60231bd996fe00bbbb375f0b9b6
comment:10 Changed 14 years ago by slavazanko
- Votes for changeset changed from angel_il to angel_il slavazanko
- severity changed from on review to approved
comment:11 Changed 14 years ago by angel_il
- Votes for changeset changed from angel_il slavazanko to committed-master
- severity changed from approved to merged
comment:12 Changed 14 years ago by angel_il
- Status changed from testing to reopened
- Resolution fixed deleted
comment:13 Changed 14 years ago by zaytsev
- Status changed from reopened to closed
- Resolution set to fixed
comment:14 Changed 14 years ago by zaytsev
- Status changed from closed to reopened
- Resolution fixed deleted
It's a stable candidate, sorry for closing prematurely.
comment:15 Changed 14 years ago by angel_il
- Votes for changeset committed-master deleted
- severity changed from merged to on review
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").
comment:17 Changed 14 years ago by zaytsev
- Cc zaytsev added
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.
comment:18 Changed 14 years ago by andrew_b
- Status changed from reopened to closed
- Resolution set to fixed
comment:19 Changed 14 years ago by andrew_b
- Votes for changeset set to committed-master
- Description modified (diff)
- severity changed from on review to merged
comment:20 Changed 14 years ago by andrew_b
- Keywords stable-candidate removed
- Votes for changeset changed from committed-master to committed-master committed-stable
changeset:23135258ef5d8f9e6d88290c8598495e914ba102 in 4.7.0-stable branch.
comment:21 Changed 11 years ago by ossi
- Description modified (diff)
- Branch state set to no branch
- Reporter changed from slavazanko to roffermanns
I really vote for getting rid of glib.
Instead we should write some tiny lib with the few primary things needed from glib.