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

make distclean: Don't delete $SAGE_ROOT/local #21775

Open
mkoeppe opened this issue Oct 27, 2016 · 7 comments
Open

make distclean: Don't delete $SAGE_ROOT/local #21775

mkoeppe opened this issue Oct 27, 2016 · 7 comments

Comments

@mkoeppe
Copy link
Contributor

mkoeppe commented Oct 27, 2016

#21501, #21479 turn SAGE_LOCAL into a configurable install hierarchy.
Consequently, $SAGE_ROOT/local should not be deleted by make distclean.

CC: @jdemeyer @dimpase @vbraun @jhpalmieri

Component: build

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

@mkoeppe mkoeppe added this to the sage-7.5 milestone Oct 27, 2016
@jdemeyer
Copy link

comment:1

I would argue that $SAGE_LOCAL should not be deleted by make distclean but that $SAGE_ROOT/local should be deleted by make distclean.

@jdemeyer

This comment has been minimized.

@jdemeyer jdemeyer changed the title make distclean: Don't delete the install hierarchy (SAGE_LOCAL) make distclean: Don't delete $SAGE_ROOT/local Oct 27, 2016
@mkoeppe
Copy link
Contributor Author

mkoeppe commented Oct 27, 2016

comment:3

That's certainly a defendable position.

What if the user explicitly used --prefix=$SAGE_ROOT/local
or to another subdirectory of $SAGE_ROOT?

@embray
Copy link
Contributor

embray commented Mar 26, 2018

comment:4

I would lean toward make distclean removes any $SAGE_LOCAL that's a subdirectory of $SAGE_ROOT. That's the point of make distclean. There's not much use for a custom --prefix outside installing Sage outside of $SAGE_ROOT.

@mkoeppe
Copy link
Contributor Author

mkoeppe commented May 1, 2020

comment:5

Moving some tickets to 9.2. This is not a promise that I will be working on them.

@mkoeppe mkoeppe modified the milestones: sage-7.5, sage-9.2 May 1, 2020
@mkoeppe mkoeppe modified the milestones: sage-9.2, sage-9.3 Aug 13, 2020
@mkoeppe
Copy link
Contributor Author

mkoeppe commented Nov 1, 2020

comment:7

I think it's time to clean up the various -clean targets in the top-level Makefile so that they match autotools behavior

@mkoeppe
Copy link
Contributor Author

mkoeppe commented Feb 13, 2021

comment:8

Setting new milestone based on a cursory review of ticket status, priority, and last modification date.

@mkoeppe mkoeppe modified the milestones: sage-9.3, sage-9.4 Feb 13, 2021
@mkoeppe mkoeppe modified the milestones: sage-9.4, sage-9.5 Jul 19, 2021
@mkoeppe mkoeppe modified the milestones: sage-9.5, sage-9.6 Dec 27, 2021
@mkoeppe mkoeppe modified the milestones: sage-9.6, sage-9.7 Apr 2, 2022
@mkoeppe mkoeppe modified the milestones: sage-9.7, sage-9.8 Aug 31, 2022
@mkoeppe mkoeppe removed this from the sage-9.8 milestone Jan 29, 2023
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