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

Developer's Guide: Update git installation instructions #30201

Closed
mkoeppe opened this issue Jul 22, 2020 · 8 comments · Fixed by #37309
Closed

Developer's Guide: Update git installation instructions #30201

mkoeppe opened this issue Jul 22, 2020 · 8 comments · Fixed by #37309

Comments

@mkoeppe
Copy link
Contributor

mkoeppe commented Jul 22, 2020

https://doc.sagemath.org/html/en/developer/git_setup.html#installing-git

On mac, it should refer to Xcode.
The linux install instructions should be autogenerated.

Depends on #29655

CC: @jhpalmieri @orlitzky @fchapoton @dimpase @Etn40ff

Component: documentation

Keywords: sd111

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

@mkoeppe mkoeppe added this to the sage-9.2 milestone Jul 22, 2020
@dimpase
Copy link
Member

dimpase commented Aug 27, 2020

comment:2

I don't quite understand what linux install instructions should be autogenerated mean in this context.

@jhpalmieri
Copy link
Member

comment:3

I'm guessing it means that for example in

Debian / Ubuntu
    sudo apt-get install git-core

the second line is constructed from something like build/pkgs/git/distros/debian.txt, or something along those lines.

@mkoeppe
Copy link
Contributor Author

mkoeppe commented Aug 27, 2020

comment:5

Yes, that's what I meant - see also #29557

@mkoeppe mkoeppe modified the milestones: sage-9.2, sage-9.3 Oct 24, 2020
@mkoeppe
Copy link
Contributor Author

mkoeppe commented Dec 6, 2020

comment:7

Hoping we can make progress on this ticket this week - https://wiki.sagemath.org/days111

@mkoeppe
Copy link
Contributor Author

mkoeppe commented Dec 6, 2020

Changed keywords from none to sd111

@mkoeppe
Copy link
Contributor Author

mkoeppe commented Jan 16, 2021

Dependencies: #29655

@mkoeppe
Copy link
Contributor Author

mkoeppe commented Jan 16, 2021

comment:9

After #29655, we can just link to the spkg page of git in the reference manual

@mkoeppe
Copy link
Contributor Author

mkoeppe commented Apr 7, 2021

comment:10

Sage development has entered the release candidate phase for 9.3. Setting a new milestone for this ticket based on a cursory review.

@mkoeppe mkoeppe modified the milestones: sage-9.3, sage-9.4 Apr 7, 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 18, 2021
@mkoeppe mkoeppe modified the milestones: sage-9.6, sage-9.7 May 3, 2022
@mkoeppe mkoeppe modified the milestones: sage-9.7, sage-9.8 Sep 19, 2022
@mkoeppe mkoeppe modified the milestones: sage-9.8, sage-9.9 Jan 7, 2023
@mkoeppe mkoeppe modified the milestones: sage-10.0, sage-10.1 Apr 30, 2023
@mkoeppe mkoeppe removed this from the sage-10.1 milestone Aug 7, 2023
vbraun pushed a commit to vbraun/sage that referenced this issue Mar 24, 2024
… the installation guide

    
<!-- ^^^^^
Please provide a concise, informative and self-explanatory title.
Don't put issue numbers in there, do this in the PR body below.
For example, instead of "Fixes sagemath#1234" use "Introduce new method to
calculate 1+1"
-->
<!-- Describe your changes here in detail -->
As proposed in https://groups.google.com/g/sage-
devel/c/5kmxaw105lg/m/liCPKG4bAQAJ

The README.md now starts with instructions to install git and then use
`git clone`.

- Also fixes sagemath#30201

<!-- Why is this change required? What problem does it solve? -->
<!-- If this PR resolves an open issue, please link to it here. For
example "Fixes sagemath#12345". -->
<!-- If your change requires a documentation PR, please link it
appropriately. -->

### 📝 Checklist

<!-- Put an `x` in all the boxes that apply. -->
<!-- If your change requires a documentation PR, please link it
appropriately -->
<!-- If you're unsure about any of these, don't hesitate to ask. We're
here to help! -->
<!-- Feel free to remove irrelevant items. -->

- [x] The title is concise, informative, and self-explanatory.
- [ ] The description explains in detail what this PR is about.
- [x] I have linked a relevant issue or discussion.
- [ ] I have created tests covering the changes.
- [ ] I have updated the documentation accordingly.

### ⌛ Dependencies

<!-- List all open PRs that this PR logically depends on
- sagemath#12345: short description why this is a dependency
- sagemath#34567: ...
-->

<!-- If you're unsure about any of these, don't hesitate to ask. We're
here to help! -->
    
URL: sagemath#37309
Reported by: Matthias Köppe
Reviewer(s): Dima Pasechnik, John H. Palmieri, Matthias Köppe
@mkoeppe mkoeppe added this to the sage-10.4 milestone Mar 25, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging a pull request may close this issue.

3 participants