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

Issue 575540045: GUILE2: generate internals doc in UTF-8 (Closed)

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

Description

GUILE2: generate internals doc in UTF-8 Otherwise, GUILE guesses Latin-1 leading to: $ isutf8 internals.texi internals.texi: line 20310, char 1, byte offset 50: invalid UTF-8 code

Patch Set 1 #

Total comments: 2

Patch Set 2 : (guile-v2) #

Unified diffs Side-by-side diffs Delta from patch set Stats (+4 lines, -0 lines) Patch
M scm/documentation-generate.scm View 1 1 chunk +4 lines, -0 lines 0 comments Download

Messages

Total messages: 5
dak
https://codereview.appspot.com/575540045/diff/571410043/scm/documentation-generate.scm File scm/documentation-generate.scm (right): https://codereview.appspot.com/575540045/diff/571410043/scm/documentation-generate.scm#newcode98 scm/documentation-generate.scm:98: (if (defined? 'set-port-encoding!) I think we usually do this ...
4 years, 3 months ago (2020-01-24 11:08:52 UTC) #1
hanwenn
(guile-v2)
4 years, 3 months ago (2020-01-24 11:14:49 UTC) #2
hanwenn
https://codereview.appspot.com/575540045/diff/571410043/scm/documentation-generate.scm File scm/documentation-generate.scm (right): https://codereview.appspot.com/575540045/diff/571410043/scm/documentation-generate.scm#newcode98 scm/documentation-generate.scm:98: (if (defined? 'set-port-encoding!) On 2020/01/24 11:08:51, dak wrote: > ...
4 years, 3 months ago (2020-01-24 11:15:10 UTC) #3
hanwenn
On 2020/01/24 11:15:10, hanwenn wrote: > https://codereview.appspot.com/575540045/diff/571410043/scm/documentation-generate.scm > File scm/documentation-generate.scm (right): > > https://codereview.appspot.com/575540045/diff/571410043/scm/documentation-generate.scm#newcode98 > ...
4 years, 3 months ago (2020-01-27 08:23:41 UTC) #4
dak
4 years, 3 months ago (2020-01-27 09:51:39 UTC) #5
On 2020/01/27 08:23:41, hanwenn wrote:
> On 2020/01/24 11:15:10, hanwenn wrote:
> >
>
https://codereview.appspot.com/575540045/diff/571410043/scm/documentation-gen...
> > File scm/documentation-generate.scm (right):
> > 
> >
>
https://codereview.appspot.com/575540045/diff/571410043/scm/documentation-gen...
> > scm/documentation-generate.scm:98: (if (defined? 'set-port-encoding!)
> > On 2020/01/24 11:08:51, dak wrote:
> > > I think we usually do this as
> > > 
> > > (if (guile-v2) ...
> > > 
> > > in order to better be able to find dependencies and ultimately (after
> > migration)
> > > throw them out.  Also I am not sure that the byte-compiler is happy about
> > those
> > > constructs.
> > 
> > Done.
> 
> ping.

I now set status of issue #5683 back to Patch-new to trigger retesting.  If you
submit followup patches via git-cl, that should happen automatically.
Sign in to reply to this message.

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