Ticket #1426 (closed defect: fixed)

Opened 7 years ago

Last modified 6 years ago

linking with --with-glib-static option problem

Reported by: zaytsev Owned by: iNode
Priority: major Milestone: 4.7.0-pre2
Component: mc-core Version: 4.6.2
Keywords: Cc:
Blocked By: Blocking:
Branch state: Votes for changeset: committed-master

Description (last modified by iNode) (diff)

(was: RHEL 4 linking problem)

Hi!

I am working on a new universal SPEC for all RH-based distros and to date I came up with

http://nopaste.com/p/aAdSlProT

Now I get

http://nopaste.com/p/auCMzt4cz

Although it compiles fine. The question is what else do I need to link against my glib statically apart from the PKG_CONFIG_PATH?

Thanks!

Attachments

txt (9.5 KB) - added by zaytsev 7 years ago.
mc.spec.in
txt.1 (266.0 KB) - added by zaytsev 7 years ago.
build.log

Change History

Changed 7 years ago by zaytsev

mc.spec.in

Changed 7 years ago by zaytsev

build.log

comment:1 Changed 7 years ago by iNode

Yet another use case for this ticket - #1436.
Fail also in ubuntu 8.04 LTS and ubuntu 9.04.

comment:2 Changed 7 years ago by iNode

  • Owner set to iNode
  • Status changed from new to accepted
  • Description modified (diff)
  • Summary changed from RHEL 4 linking problem to linking with --with-glib-static option problem

library search order change proposed, it should fix this problem,
please try:

-- src/Makefile.am.orig	2009-07-28 10:43:41.000000000 +0400
+++ src/Makefile.am	2009-08-01 16:28:53.000000000 +0400
@@ -42,4 +42,4 @@
 mc_LDADD = $(EDITLIB) $(SLANGLIB) $(VFSLIB) \
-	$(INTLLIBS) $(GLIB_LIBS) $(MCLIBS) $(LIBICONV) search/libsearch.la \
-	mcconfig/libmcconfig.la
+	$(INTLLIBS) $(MCLIBS) search/libsearch.la \
+	mcconfig/libmcconfig.la $(GLIB_LIBS) $(LIBICONV)

comment:3 follow-up: ↓ 4 Changed 7 years ago by iNode

  • severity changed from no branch to on review

branch created: 98f0fd1ca7608ac1a42c969892038895e270b80a

please test and review

comment:4 in reply to: ↑ 3 ; follow-up: ↓ 5 Changed 7 years ago by iNode

Replying to iNode:

branch created: 98f0fd1ca7608ac1a42c969892038895e270b80a

please test and review

another branch with correct name: 3d6552894fb17781a7f2e3d9172e55e99a553e1f

comment:5 in reply to: ↑ 4 Changed 7 years ago by iNode

additional cleanup changeset

comment:6 Changed 7 years ago by zaytsev

  • Votes for changeset set to zaytsev

Now works for me.

comment:7 Changed 7 years ago by slyfox

  • Votes for changeset changed from zaytsev to zaytsev slyfox

Same here: now works correctly (gentoo amd64)

comment:8 Changed 7 years ago by angel_il

  • Votes for changeset changed from zaytsev slyfox to zaytsev slyfox angel_il
  • severity changed from on review to approved

comment:9 Changed 7 years ago by angel_il

  • Votes for changeset changed from zaytsev slyfox angel_il to zaytsev slyfox
  • severity changed from approved to on rework

/home/projects/mc/src/mcconfig/set.c:85: undefined reference to `g_key_file_set_boolean'
mcconfig/.libs/libmcconfig.a(libmcconfig_la-set.o): In function `mc_config_set_string':
/home/projects/mc/src/mcconfig/set.c:72: undefined reference to `g_key_file_set_string'

comment:10 Changed 7 years ago by angel_il

  • Votes for changeset changed from zaytsev slyfox to zaytsev slyfox angel_il
  • severity changed from on rework to approved

comment:11 Changed 7 years ago by iNode

  • Status changed from accepted to testing
  • Votes for changeset changed from zaytsev slyfox angel_il to commited master
  • Resolution set to fixed
  • severity changed from approved to merged

comment:12 Changed 7 years ago by iNode

  • Votes for changeset changed from commited master to commited-master

comment:13 Changed 7 years ago by zaytsev

  • Status changed from testing to closed
Note: See TracTickets for help on using tickets.