Opened 8 years ago

Closed 8 years ago

Last modified 8 years ago

#11816 closed defect (fixed)

Typo in the developer's guide

Reported by: saraedum Owned by: mvngu
Priority: trivial Milestone: sage-4.7.2
Component: documentation Keywords:
Cc: Merged in: sage-4.7.2.alpha3
Authors: Julian Rueth Reviewers: Leif Leonhardy
Report Upstream: N/A Work issues:
Branch: Commit:
Dependencies: Stopgaps:

Description (last modified by leif)

Should one really open a ticket for something like this?

where <repo-name> is the name of the repository you are using, and <format is “html”, “pdf”, [...]

Apply only trac_11816_typo.proper.patch to the Sage library.

Attachments (2)

trac_11816_typo.patch (655 bytes) - added by saraedum 8 years ago.
fixes a typo in the developer's guide
trac_11816_typo.proper.patch (638 bytes) - added by leif 8 years ago.
"Proper" Mercurial changeset replacement patch.

Download all attachments as: .zip

Change History (7)

Changed 8 years ago by saraedum

fixes a typo in the developer's guide

comment:1 Changed 8 years ago by saraedum

  • Status changed from new to needs_review

comment:2 in reply to: ↑ description Changed 8 years ago by leif

  • Reviewers set to Leif Leonhardy
  • Status changed from needs_review to positive_review

Replying to saraedum:

Should one really open a ticket for something like this?

In principle yes, since otherwise they're likely to stay forever.

In this case, I'm pretty sure I've already corrected it in a not-yet-published version (a reviewer patch I have to provide for another ticket), but haven't found the time to finish it.

Once you start looking for typos, wrong formatting and inconsistencies (or meanwhile obsolete information), you're likely to not get to an end, unfortunately... ;-)

comment:3 Changed 8 years ago by leif

  • Description modified (diff)

comment:4 Changed 8 years ago by leif

  • Merged in set to sage-4.7.2.alpha3
  • Resolution set to fixed
  • Status changed from positive_review to closed

Changed 8 years ago by leif

"Proper" Mercurial changeset replacement patch.

comment:5 Changed 8 years ago by leif

  • Description modified (diff)

I've attached a *.proper.patch, which is identical except that I removed the "garbage" before "# HG changeset patch", i.e., I deleted the first line "exporting patch:", since Jeroen's current merger rejects such patches.

For now, please make sure all your patches start with "# HG changeset patch", i.e., have it on the first line without any preceding messages or whatever.

I've relaxed that in my version of the merger, but Jeroen and maybe others are likely to use his more restrictive one.

Note: See TracTickets for help on using tickets.