Opened 13 years ago

Closed 12 years ago

Last modified 12 years ago

#7337 closed defect (fixed)

PolyBoRi fails to build on cygwin

Reported by: Mike Hansen Owned by: tbd
Priority: major Milestone: sage-4.4.3
Component: porting: Cygwin Keywords:
Cc: William Stein Merged in: sage-4.4.3.alpha1
Authors: Mike Hansen Reviewers: William Stein
Report Upstream: N/A Work issues:
Branch: Commit:
Dependencies: Stopgaps:

Status badges

Description

It fails with 

  /home/mhansen/sage-4.2/spkg/build/gd-2.0.35.p2/src/gd_png.c:70: undefined reference to `_png_get_error_ptr'
  /home/mhansen/sage-4.2/local/lib/libgd.a(gd_png.o): In function `gdPngReadData':
  /home/mhansen/sage-4.2/spkg/build/gd-2.0.35.p2/src/gd_png.c:85: undefined reference to `_png_get_io_ptr'
  /home/mhansen/sage-4.2/spkg/build/gd-2.0.35.p2/src/gd_png.c:87: undefined reference to `_png_error'
  /home/mhansen/sage-4.2/local/lib/libgd.a(gd_png.o): In function `gdPngWriteData':
  /home/mhansen/sage-4.2/spkg/build/gd-2.0.35.p2/src/gd_png.c:94: undefined reference to `_png_get_io_ptr'
  /home/mhansen/sage-4.2/local/lib/libgd.a(gd_png.o): In function `gdImageCreateFromPngCtx':
  /home/mhansen/sage-4.2/spkg/build/gd-2.0.35.p2/src/gd_png.c:152: undefined reference to `_png_check_sig'

This can be fixed by adding png12 and z to the list of libraries needed when gd is present.

I'll post an spkg shortly.

Change History (5)

comment:1 Changed 13 years ago by Mike Hansen

Milestone: → sage-duplicate/invalid/wontfix
Report Upstream: → N/A
Resolution: → invalid
Status: new → closed

This no longer fails with Cygwin 1.7.

comment:2 Changed 12 years ago by Mike Hansen

Resolution: invalid
Status: closed → new

So, it turns out this is still an issue.

comment:3 Changed 12 years ago by Mike Hansen

Authors: → Mike Hansen
Status: new → needs_review

There's an spkg which should fix this at ​http://sage.math.washington.edu/home/mhansen/polybori-0.6.4.p1.spkg

comment:4 Changed 12 years ago by William Stein

Merged in: → 4.4.3.alpha1
Resolution: → fixed
Reviewers: → wstein
Status: needs_review → closed

looks good

comment:5 Changed 12 years ago by Minh Van Nguyen

Merged in: 4.4.3.alpha1 → sage-4.4.3.alpha1
Milestone: sage-duplicate/invalid/wontfix → sage-4.4.3
Reviewers: wstein → William Stein
Note: See TracTickets for help on using tickets.