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

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

Can't Edit
Can't Publish+Mail
Start Review
Created:
1 month, 2 weeks ago by hahnjo
Modified:
1 month ago
Reviewers:
fedelogy
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: 4
hahnjo
As I'm not doing translations myself, it would be great if somebody familiar with the ...
1 month, 2 weeks 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 ...
1 month, 2 weeks ago (2020-04-20 09:09:22 UTC) #2
hahnjo
delete texi-skeleton-update.py and update documentation
1 month, 2 weeks ago (2020-04-20 17:20:01 UTC) #3
hahnjo
1 month, 2 weeks ago (2020-04-20 17:24:41 UTC) #4
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