Rietveld Code Review Tool
Help | Bug tracker | Discussion group | Source code | Sign in
(76)

Issue 545910043: Documentation: Remove deprecated skeleton-update (Closed)

Can't Edit
Can't Publish+Mail
Start Review
Created:
4 years ago by hahnjo
Modified:
3 years, 2 months ago
CC:
lilypond-devel_gnu.org
Visibility:
Public.

Description

Documentation: Remove deprecated skeleton-update It's been like that since commit 858e506c2a in 2010, ten years ago. Also remove target all-translations-update which cannot have worked since the same time and update some CG pages mentioning the targets and the underlying script texi-skeleton-update.py. Rewrite the section about 'Adding a Texinfo manual' because it was out-of-date with respect to the build system: Directories with the included .itely files are not listed in SUBDIRS and have no GNUmakefile that would need to be copied.

Patch Set 1 #

Patch Set 2 : delete texi-skeleton-update.py and update documentation #

Unified diffs Side-by-side diffs Delta from patch set Stats (+5 lines, -75 lines) Patch
M Documentation/GNUmakefile View 2 chunks +0 lines, -11 lines 0 comments Download
M Documentation/contributor/doc-work.itexi View 1 4 chunks +5 lines, -39 lines 0 comments Download
D scripts/auxiliar/texi-skeleton-update.py View 1 1 chunk +0 lines, -25 lines 0 comments Download

Messages

Total messages: 5
hahnjo
As I'm not doing translations myself, it would be great if somebody familiar with the ...
4 years ago (2020-04-19 15:04:38 UTC) #1
fedelogy
On 2020/04/19 15:04:38, hahnjo wrote: > As I'm not doing translations myself, it would be ...
4 years ago (2020-04-20 09:09:22 UTC) #2
hahnjo
delete texi-skeleton-update.py and update documentation
4 years ago (2020-04-20 17:20:01 UTC) #3
hahnjo
On 2020/04/20 09:09:22, fedelogy wrote: > On 2020/04/19 15:04:38, hahnjo wrote: > > As I'm ...
4 years ago (2020-04-20 17:24:41 UTC) #4
vazquezfajardo.juanalberto810
3 years, 2 months ago (2021-02-01 14:46:35 UTC) #5
Message was sent while issue was closed.
On 2020/04/20 17:24:41, hahnjo wrote:
> On 2020/04/20 09:09:22, fedelogy wrote:
> > On 2020/04/19 15:04:38, hahnjo wrote:
> > > As I'm not doing translations myself, it would be great if somebody
familiar
> > > with the process could comment on the faith of
> > > scripts/auxiliar/texi-skeleton-update.py: It's not failing, but to me
> doesn't
> > > look like doing anything helpful either.
> > 
> > I remember I tried using it when I started the translation about 10 years
ago
> > and it wasn't helpful at all.
> > I'm Ok with removing it. Other translators will hopefully confirm (or not).
> 
> I've removed it in the updated patch set. If somebody needs it after this
patch
> landed (which I'll delay a bit to give a fair chance for objection), we can
> still restore it from the git history.
> 
> > However, I would not remove completely that section in the CG.
> > In general, the documentation doesn't take anything for granted, especially
> when
> > information is addressed to newcomers.
> > I would write something like:
> > 
> > """
> > Adding a Texinfo manual
> > 
> > In order to start translating a new manual, simply copy the english files
> within
> > your language directory and translate them.
> > For example, if you want to translate the first chapter of the Learning
> Manual:
> > 
> > cp Documentation/learning.tely Documentation/LANG/learning.tely
> > cp Documentation/learning/tutorial.itely Documentation/LANG/tutorial.itely
> > """
> 
> Ah, that matches more closely what I would have expected. I've added your text
> to the patch, thanks!
Sign in to reply to this message.

Powered by Google App Engine
RSS Feeds Recent Issues | This issue
This is Rietveld f62528b