Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Singular does not build on Ubuntu Oneiric #11769

Closed
dandrake opened this issue Sep 1, 2011 · 12 comments
Closed

Singular does not build on Ubuntu Oneiric #11769

dandrake opened this issue Sep 1, 2011 · 12 comments

Comments

@dandrake
Copy link
Contributor

dandrake commented Sep 1, 2011

Due to fussier linkers, Singular will not build on the current version of Ubuntu Oneiric (11.10). This was reported at https://groups.google.com/d/topic/sage-devel/aL8b9phz3Xc/discussion:

../kernel/libkernel.a(mod_raw.o): In function `dynl_open':
/home/foo/sage-4.7.2.alpha2/spkg/build/singular-3-1-1-4.p13/src/kernel/mod_raw.cc:217: undefined reference to `dlopen'
../kernel/libkernel.a(mod_raw.o): In function `dynl_sym':
/home/foo/sage-4.7.2.alpha2/spkg/build/singular-3-1-1-4.p13/src/kernel/mod_raw.cc:228: undefined reference to `dlsym'
../kernel/libkernel.a(mod_raw.o): In function `dynl_close':
/home/foo/sage-4.7.2.alpha2/spkg/build/singular-3-1-1-4.p13/src/kernel/mod_raw.cc:233: undefined reference to `dlclose'
../kernel/libkernel.a(mod_raw.o): In function `dynl_error':
/home/foo/sage-4.7.2.alpha2/spkg/build/singular-3-1-1-4.p13/src/kernel/mod_raw.cc:238: undefined reference to `dlerror'
collect2: ld returned 1 exit status
make[2]: *** [iparith.inc] Error 1
make[2]: Leaving directory `/home/foo/sage-4.7.2.alpha2/spkg/build/singular-3-1-1-4.p13/src/Singular'
make[1]: *** [install] Error 1
make[1]: Leaving directory `/home/foo/sage-4.7.2.alpha2/spkg/build/singular-3-1-1-4.p13/src'
make: *** [/home/foo/sage-4.7.2.alpha2/local/bin/Singular-3-1-1] Error 2
Unable to build Singular.

(Cf. also [comment:ticket:11674:27] ff.)

The fix is to flip the order of -ldl and -lkernel, or append a second -ldl in the linking steps.

This may already be fixed upstream (cf. #10903); if so, this ticket should be closed. If not, the Sage/Singular Days (SD34) would be the perfect time to fix it.

According to Hannes, this is now fixed in svn revision 14381, i.e., not yet in any stable upstream release, so we should incorporate at least that patch into some new Singular spkg.

Fixed spkg to be found at #10903.

Depends on #10903

Upstream: Workaround found; Bug reported upstream.

CC: @burcin

Component: packages: standard

Keywords: singular, linker, sd34 -ldl libkernel mod_raw.cc

Reviewer: Volker Braun

Issue created by migration from https://trac.sagemath.org/ticket/11769

@nexttime

This comment has been minimized.

@nexttime
Copy link
Mannequin

nexttime mannequin commented Sep 1, 2011

Changed keywords from singular, linker, sd34 to singular, linker, sd34 -ldl libkernel mod_raw.cc

@nexttime
Copy link
Mannequin

nexttime mannequin commented Sep 2, 2011

Upstream: Workaround found; Bug reported upstream.

@nexttime
Copy link
Mannequin

nexttime mannequin commented Sep 2, 2011

This may already be fixed upstream (cf. #10903); if so, this ticket should be closed. If not, the Sage/Singular Days (SD34) would be the perfect time to fix it.

It is not fixed in Singular 3-1-3-2 (current latest stable), nor in the current svn revision 14378.

Upstream report: http://www.singular.uni-kl.de:8002/trac/ticket/368

@nexttime
Copy link
Mannequin

nexttime mannequin commented Sep 11, 2011

comment:3

For the record: So far nothing happened on the upstream ticket.

@nexttime nexttime mannequin added p: critical / 2 and removed p: major / 3 labels Sep 11, 2011
@nexttime
Copy link
Mannequin

nexttime mannequin commented Sep 14, 2011

comment:5

Replying to @nexttime:

For the record: So far nothing happened on the upstream ticket.

According to upstream, they've now reluctantly fixed this, in svn revision 14381. (Haven't checked this yet.)

@nexttime

This comment has been minimized.

@nexttime
Copy link
Mannequin

nexttime mannequin commented Sep 24, 2011

comment:7

Making this a blocker, since IMHO at least a work-around should be in the final Sage 4.7.2, likely to be released [right] before the final Ubuntu 11.10 (Oneiric Ocelot).

If we don't get a new Singular spkg fixing this from SD34, we should provide an spkg just adding -ldl to LDFLAGS (on Linux only), or applying the upstream fix from the svn. (Haven't tried the latter yet; should IMHO work, despite the age of our current Singular version.)

@nexttime nexttime mannequin added p: blocker / 1 and removed p: critical / 2 labels Sep 24, 2011
@vbraun
Copy link
Member

vbraun commented Oct 2, 2011

Dependencies: #10903

@vbraun
Copy link
Member

vbraun commented Oct 2, 2011

comment:8

The new Singular-3-1-3-3 spkg from #10903 links with -lkernel in front of -ldl:

g++ -O2 -g -fPIC -I.. -I/home/vbraun/Sage/sage/local -pipe -I. -I.. -I/home/vbraun/Sage/sage/local -I/h
ome/vbraun/Sage/sage/local/include -I/home/vbraun/Sage/sage/local/include -I/home/vbraun/Sage/sage/loca
l/include   -fno-implicit-templates -I.. -I/home/vbraun/Sage/sage/local -DNDEBUG -DOM_NDEBUG -Dx86_64_L
inux -DHAVE_CONFIG_H \
  -o Singular \
  tesths.cc iparith.o mpsr_Tok.o claptmpl.o\
  grammar.o scanner.o attrib.o blackbox.o eigenval_ip.o extra.o fehelp.o feOpt.o ipassign.o ipconv.o ipid.o iplib.o ipprint.o ipshell.o newstruct.o lists.o sdb.o fglm.o interpolation.o silink.o ssiLink.o subexpr.o janet.o wrapper.o libparse.o sing_win.o gms.o pcv.o maps_ip.o walk.o walk_ip.o cntrlc.o misc_ip.o calcSVD.o pipeLink.o Minor.o MinorProcessor.o MinorInterface.o bigintm.o pyobject_setup.o bbcone.o bbfan.o denom_list.o minpoly.o  slInit_Dynamic.o -rdynamic -L/home/vbraun/Sage/sage/local/kernel -L../kernel -lkernel -L/home/vbraun/Sage/sage/local/lib -L/home/vbraun/Sage/sage/local/lib   -ldl -lm -lsingfac -lsingcf -lntl -lgmp -lreadline -ltermcap -lm  -lnsl -lbsd  -lomalloc  ../kernel/mmalloc.o 

So this issue is solved by its dependency. I will set this ticket to positive review.

@vbraun
Copy link
Member

vbraun commented Oct 2, 2011

Reviewer: Volker Braun

@jdemeyer

This comment has been minimized.

@jdemeyer jdemeyer closed this as completed Oct 4, 2011
@jdemeyer jdemeyer removed this from the sage-4.7.2 milestone Oct 4, 2011
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

3 participants