From mboxrd@z Thu Jan 1 00:00:00 1970 From: Nicolas Goaziou Subject: Re: [PATCH] Add new keyword :coding for #+include directive Date: Sat, 02 Jun 2018 12:13:01 +0200 Message-ID: <876031o5si.fsf@nicolasgoaziou.fr> References: <87tvsbx72g.fsf@killashandra.ballybran.fr> <87in8qyh3t.fsf@killashandra.ballybran.fr> <874lkakz58.fsf@nicolasgoaziou.fr> <87muy0b4kw.fsf@killashandra.ballybran.fr> <87in8lh3xk.fsf@killashandra.ballybran.fr> <87zi1uqkta.fsf@nicolasgoaziou.fr> <87in8hd7xe.fsf@killashandra.ballybran.fr> <87zi1sl0yq.fsf@nicolasgoaziou.fr> <874lk01ad6.fsf@killashandra.ballybran.fr> <871serrqla.fsf@killashandra.ballybran.fr> <87o9hqrr54.fsf@nicolasgoaziou.fr> <87y3gl6bw7.fsf@killashandra.ballybran.fr> <87d0xrhjp7.fsf@nicolasgoaziou.fr> <87po1rocer.fsf@killashandra.ballybran.fr> <87y3geg4sh.fsf@nicolasgoaziou.fr> <87a7sooi87.fsf@killashandra.ballybran.fr> Mime-Version: 1.0 Content-Type: text/plain; charset=utf-8 Content-Transfer-Encoding: quoted-printable Return-path: Received: from eggs.gnu.org ([2001:4830:134:3::10]:53442) by lists.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1fP3X3-0006vS-3B for emacs-orgmode@gnu.org; Sat, 02 Jun 2018 06:13:09 -0400 Received: from Debian-exim by eggs.gnu.org with spam-scanned (Exim 4.71) (envelope-from ) id 1fP3Wz-00032g-T3 for emacs-orgmode@gnu.org; Sat, 02 Jun 2018 06:13:09 -0400 Received: from relay5-d.mail.gandi.net ([217.70.183.197]:37153) by eggs.gnu.org with esmtps (TLS1.0:DHE_RSA_AES_256_CBC_SHA1:32) (Exim 4.71) (envelope-from ) id 1fP3Wz-00031I-MA for emacs-orgmode@gnu.org; Sat, 02 Jun 2018 06:13:05 -0400 In-Reply-To: <87a7sooi87.fsf@killashandra.ballybran.fr> ("Pierre =?utf-8?Q?T=C3=A9choueyres=22's?= message of "Thu, 24 May 2018 23:29:28 +0200") List-Id: "General discussions about Org-mode." List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Errors-To: emacs-orgmode-bounces+geo-emacs-orgmode=m.gmane.org@gnu.org Sender: "Emacs-orgmode" To: Pierre =?utf-8?Q?T=C3=A9choueyres?= Cc: org-mode Hello, pierre.techoueyres@free.fr (Pierre T=C3=A9choueyres) writes: > Hello Nicolas, > I'm sorry for the delay, this problem drove me crazy for a few days. > So I started again from the beginning and, of course, I am now unable to > replicate my original test case. > So, I think you could delete the > 0001-Manage-the-encoding-of-files-with-include-.patch patch. > I am really sorry for the inconvenience. No problem. It also drove me crazy. > I hope we can go ahead with the other patch to add a ":coding" keyword > to the include directive. > > To resume the goal: > The include keyword works well with encodings like cp850 if, and only > if, there are some indications of the encoding in the file (as local > variables for example). But sometimes, there is no such information and > you can not modify the file (for various reasons. ex: the file is > generated automatically). I applied the first patch in "next" branch, with a slight refactoring. Thank you. Regards, --=20 Nicolas Goaziou 0x80A93738