zlib should be a prerequisite for libpng

[jhpalmieri]

From the libpng INSTALL file:

Before installing libpng, you must first install zlib

So fix the file "deps" accordingly. (This issue arose when building spkg's in parallel: the build failed when it reached libpng because zlib wasn't installed yet.)

CC: @sagetrac-drkirkby

Component: distribution

Author: John Palmieri

Reviewer: David Kirkby

Merged: sage-4.5.2.alpha1

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

[sagetrac-drkirkby]

comment:1

That's perfectly logical. Positive review.

[jhpalmieri]

comment:3

I'm upgrading this to a blocker because there is a patch and I hit it every time I try to build sage on the skynet machine iras with parallel spkg building: the build fails on libpng, then I have to restart it. (Usually libpng and zlib are built at the same time on this machine, so after libpng fails, zlib builds successfully, so running "make" again works fine.)

[dandrake]

comment:4

I've put the deps file here in SAGE_ROOT/spkg/standard for 4.5.2.alpha1.

[dandrake]

Reviewer: David Kirkby

[dandrake]

Merged: sage-4.5.2.alpha1

[dandrake]

Author: John Palmieri

[sagetrac-drkirkby]

comment:5

I'm sorry, I have screwed up here, and attached these files to the wrong ticket! Instead of attaching them to #9598, I've attached them here.

Since this has not been merged, it's probably not a fatal screw up. Sorry.

I will attempt to put the files back as they should be.

[sagetrac-drkirkby]

comment:6

Replying to @sagetrac-drkirkby:

Since this has not been merged, it's probably not a fatal screw up. Sorry.

I mean it has been merged, so it should not be a big deal. But I will attempt to put the files back as they should be.

[sagetrac-drkirkby]

comment:7

I've tried to correct this, but have failed. I'm going to delete this, and raise it on sage-devel.

Dave

[sagetrac-drkirkby]

Attachment: deps.gz

spkg/standard/deps file, written by John and merged into sage-4.5.2.alpha1 which I overwrote by mistake. I'm 99% sure this is right.

[sagetrac-drkirkby]

Attachment: deps.diff.gz

diff of spkg/standard/deps written by john which I overwrote by mistake. I'm 99% sure this is right.

[sagetrac-drkirkby]

comment:8

I believe these are correct. Perhaps someone else can check.

Dave