Groups | Search | Server Info | Keyboard shortcuts | Login | Register [http] [https] [nntp] [nntps]


Groups > linux.kernel > #1722044 > unrolled thread

[PATCH 1/2] docs: kernel-doc comments are ASCII

Started byRandy Dunlap <rdunlap@infradead.org>
First post2017-08-29 01:20 +0200
Last post2017-08-31 18:00 +0200
Articles 1 on this page of 21 — 4 participants

Back to article view | Back to linux.kernel


Contents

  [PATCH 1/2] docs: kernel-doc comments are ASCII Randy Dunlap <rdunlap@infradead.org> - 2017-08-29 01:20 +0200
    Re: [PATCH 1/2] docs: kernel-doc comments are ASCII Jonathan Corbet <corbet@lwn.net> - 2017-08-30 23:30 +0200
      Re: [PATCH 1/2] docs: kernel-doc comments are ASCII Randy Dunlap <rdunlap@infradead.org> - 2017-08-31 00:10 +0200
        Re: [PATCH 1/2] docs: kernel-doc comments are ASCII Mauro Carvalho Chehab <mchehab@s-opensource.com> - 2017-08-31 00:20 +0200
          Re: [PATCH 1/2] docs: kernel-doc comments are ASCII Jonathan Corbet <corbet@lwn.net> - 2017-08-31 00:40 +0200
            Re: [PATCH 1/2] docs: kernel-doc comments are ASCII Randy Dunlap <rdunlap@infradead.org> - 2017-08-31 01:10 +0200
              Re: [PATCH 1/2] docs: kernel-doc comments are ASCII Randy Dunlap <rdunlap@infradead.org> - 2017-08-31 01:10 +0200
                Re: [PATCH 1/2] docs: kernel-doc comments are ASCII Randy Dunlap <rdunlap@infradead.org> - 2017-08-31 01:20 +0200
        Re: [PATCH 1/2] docs: kernel-doc comments are ASCII Mauro Carvalho Chehab <mchehab@s-opensource.com> - 2017-08-31 12:00 +0200
          Re: [PATCH 1/2] docs: kernel-doc comments are ASCII Jani Nikula <jani.nikula@linux.intel.com> - 2017-08-31 15:30 +0200
            Re: [PATCH 1/2] docs: kernel-doc comments are ASCII Mauro Carvalho Chehab <mchehab@s-opensource.com> - 2017-08-31 16:00 +0200
              Re: [PATCH 1/2] docs: kernel-doc comments are ASCII Jonathan Corbet <corbet@lwn.net> - 2017-08-31 16:20 +0200
                Re: [PATCH 1/2] docs: kernel-doc comments are ASCII Randy Dunlap <rdunlap@infradead.org> - 2017-08-31 17:50 +0200
                  Re: [PATCH 1/2] docs: kernel-doc comments are ASCII Jani Nikula <jani.nikula@linux.intel.com> - 2017-08-31 18:10 +0200
                    Re: [PATCH 1/2] docs: kernel-doc comments are ASCII Randy Dunlap <rdunlap@infradead.org> - 2017-08-31 18:30 +0200
                    Re: [PATCH 1/2] docs: kernel-doc comments are ASCII Jani Nikula <jani.nikula@linux.intel.com> - 2017-08-31 18:40 +0200
                      Re: [PATCH 1/2] docs: kernel-doc comments are ASCII Randy Dunlap <rdunlap@infradead.org> - 2017-08-31 19:40 +0200
                        Re: [PATCH 1/2] docs: kernel-doc comments are ASCII Randy Dunlap <rdunlap@infradead.org> - 2017-08-31 20:00 +0200
                        Re: [PATCH 1/2] docs: kernel-doc comments are ASCII Jani Nikula <jani.nikula@linux.intel.com> - 2017-08-31 20:00 +0200
                          Re: [PATCH 1/2] docs: kernel-doc comments are ASCII Randy Dunlap <rdunlap@infradead.org> - 2017-08-31 20:10 +0200
          Re: [PATCH 1/2] docs: kernel-doc comments are ASCII Randy Dunlap <rdunlap@infradead.org> - 2017-08-31 18:00 +0200

Page 2 of 2 — ← Prev page 1 [2]


#1724373

FromRandy Dunlap <rdunlap@infradead.org>
Date2017-08-31 18:00 +0200
Message-ID<ukA2R-7ug-13@gated-at.bofh.it>
In reply to#1724054
On 08/31/17 02:49, Mauro Carvalho Chehab wrote:
> Em Wed, 30 Aug 2017 15:02:59 -0700
> Randy Dunlap <rdunlap@infradead.org> escreveu:
> 
>> On 08/30/17 14:23, Jonathan Corbet wrote:
>>> On Mon, 28 Aug 2017 16:10:09 -0700
>>> Randy Dunlap <rdunlap@infradead.org> wrote:
>>>   
>>>> kernel-doc parsing uses as ASCII codec, so let people know that
>>>> kernel-doc comments should be in ASCII characters only.
>>>>
>>>> WARNING: kernel-doc '../scripts/kernel-doc -rst -enable-lineno ../drivers/media/dvb-core/demux.h' processing failed with: 'ascii' codec can't decode byte 0xe2 in position 6368: ordinal not in range(128)  
>>>
>>> So I don't get this error.  What kind of system are you running the docs
>>> build on?  I would really rather that the docs system could handle modern
>>> text if possible, so it would be better to figure out what's going on
>>> here...  
>>
>> I'm OK with that. Source files in general don't need to be ASCII (0-127).
>>
>> I did this patch based on this (private) comment:
>>
>>> Yes, using ASCII should fix the problem.  
>>
>> what kind of system?  HP laptop.
>>
>> Linux midway.site 4.4.79-18.26-default #1 SMP Thu Aug 10 20:30:05 UTC 2017 (fa5a935) x86_64 x86_64 x86_64 GNU/Linux
>>
>>> sphinx-build --version  
>> Sphinx (sphinx-build) 1.3.1
> 
> I tried hard to reproduce the error here... I even added some Chinese
> chars on a kernel-doc markup and changed the language on my system
> to LANG=en_US.iso885915.
> 
> No luck. 
> 
> As Documentation/conf.py has:
> 
> 	# -*- coding: utf-8 -*-
> 
> on its first line, I suspect that the error you're getting is likely
> due to the usage of a python version that doesn't recognize this.
> 
> It seems that such dialect was introduced on python version 2.3:
> 
> 	https://docs.python.org/2.3/whatsnew/section-encodings.html
> 
> Yet, the documentation there seems to require a line before it,
> e. g.:
> 
> 	#!/usr/bin/env python
> 	# -*- coding: UTF-8 -*-
> 
> I suspect, however, that, if such line is added, on some systems it
> may not work, e. g. if both python 2 and 3 are installed, it could
> use the python version that doesn't have Sphinx installed.
> 
> So, I suspect that the safest way to fix it is with something like the
> enclosed patch. Still, it could be useful to know what's happening,
> just in case we get other reports.
> 
> Randy,
> 
> What's your python version?

> python --version
Python 2.7.13



-- 
~Randy

[toc] | [prev] | [standalone]


Page 2 of 2 — ← Prev page 1 [2]

Back to top | Article view | linux.kernel


csiph-web