Opened 11 years ago

Last modified 11 years ago

#9681 closed defect

Missing dependancy in spkg/standard/deps for zn_poly. — at Initial Version

Reported by: drkirkby Owned by: GeorgSWeber
Priority: minor Milestone: sage-duplicate/invalid/wontfix
Component: build Keywords:
Cc: ddrake, mpatel, leif, jhpalmieri Merged in:
Authors: Reviewers:
Report Upstream: N/A Work issues:
Branch: Commit:
Dependencies: Stopgaps:

Status badges

Description

The zn_poly package lists in SPKG.txt the only dependencies are GMP, but this is not true, as zn_poly's configure script has in it:

/configure --gmp-prefix="$SAGE_LOCAL" --ntl-prefix="$SAGE_LOCAL" \
            --prefix="$SAGE_LOCAL" --cflags="$CFLAGS" --ldflags="$LDFLAGS"

Looking at $SAGE_ROOT/spkg/standard/deps, I see:

$(INST)/$(ZNPOLY): $(BASE) $(INST)/$(MPIR)
        $(INSTALL) "$(SAGE_SPKG) $(ZNPOLY) 2>&1" "tee -a $(SAGE_LOGS)/$(ZNPOLY).log"

then looking at MPIR I see the dependencies are only BASE and ICONV. But ICONV only depends on BASE, so there is nothing to force ntl to build before zn_poly.

I am aware of two other changes that are desirable in the 'deps' file too, as they add clarity.

These might as well be fixed at the same time.

Dave

Change History (0)

Note: See TracTickets for help on using tickets.