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

Put prereq under revision control #12739

Closed
jdemeyer opened this issue Mar 23, 2012 · 8 comments
Closed

Put prereq under revision control #12739

jdemeyer opened this issue Mar 23, 2012 · 8 comments

Comments

@jdemeyer
Copy link

The contents of the prereq tarball are currently not under revision control. Fix this. Only non-automatically generated files are put under revision control, these are:

C .hgignore
C .hgtags
C Makefile.am
C configure.ac
C m4/ax_c_check_flag.m4
C m4/ax_gcc_option.m4
C m4/ax_gcc_version.m4
C m4/ax_gxx_option.m4
C m4/ax_gxx_version.m4
C m4/ax_prog_perl_version.m4

Further changes:

  1. In configure.ac, comment AC_CONFIG_HEADERS([config.h]) as we don't need this header.

  2. Remove some unneeded files, in particular the autom4te.cache directory.

  3. Compress the prereq tarball using gzip. Note that gzip has always been a dependency of Sage, since the bzip2 package is gzip-compressed.

Apply attachment: 12739_prereq_tar_gz.patch to the SAGE_ROOT repository

Delete the old SAGE_ROOT/spkg/base/prereq-0.9.tar

Download http://boxen.math.washington.edu/home/jdemeyer/spkg/prereq-0.9.tar.gz to SAGE_ROOT/spkg/base

CC: @jhpalmieri

Component: packages: standard

Author: Jeroen Demeyer

Reviewer: John Palmieri

Merged: sage-5.0.beta12

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

@jdemeyer
Copy link
Author

First and only commit in the prereq tarball, for review only

@jdemeyer
Copy link
Author

comment:1

Attachment: prereq-0.9.diff.gz

@jdemeyer

This comment has been minimized.

@jdemeyer

This comment has been minimized.

@jdemeyer
Copy link
Author

comment:5

Attachment: 12739_prereq_tar_gz.patch.gz

@jhpalmieri
Copy link
Member

Reviewer: John Palmieri

@jhpalmieri
Copy link
Member

comment:7

This looks fine to me. The version should get updated soon, but I guess that happens in #12112.

@jdemeyer
Copy link
Author

jdemeyer commented Apr 2, 2012

Merged: sage-5.0.beta12

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

2 participants