From mboxrd@z Thu Jan 1 00:00:00 1970 From: Bastien Subject: Re: [PATCH] Update documentation for :exports (applicable to blocks only, not inline) Date: Fri, 14 Mar 2014 16:51:13 +0100 Message-ID: <87txb0pymm.fsf@bzg.ath.cx> References: <87y50fwidf.fsf@bzg.ath.cx> <87eh26ms19.fsf@bzg.ath.cx> Mime-Version: 1.0 Content-Type: text/plain Return-path: Received: from eggs.gnu.org ([2001:4830:134:3::10]:37707) by lists.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1WOUOQ-0004ov-8h for emacs-orgmode@gnu.org; Fri, 14 Mar 2014 11:51:36 -0400 Received: from Debian-exim by eggs.gnu.org with spam-scanned (Exim 4.71) (envelope-from ) id 1WOUOH-0008WV-Em for emacs-orgmode@gnu.org; Fri, 14 Mar 2014 11:51:30 -0400 Received: from rs249.mailgun.us ([209.61.151.249]:39267) by eggs.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1WOUOH-0008WA-7A for emacs-orgmode@gnu.org; Fri, 14 Mar 2014 11:51:21 -0400 In-Reply-To: (John Hendy's message of "Fri, 14 Mar 2014 10:44:57 -0500") 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-bounces+geo-emacs-orgmode=m.gmane.org@gnu.org To: John Hendy Cc: emacs-orgmode Hi John, thanks for the patch -- some comments below. John Hendy writes: > Header arguments: clarify that `:exports' is only applicable to > blocks, not inline code This first line should include "org.texi:" to tell that the change happens in org.texi. See other commits for examples. > * doc/org.texi (Exporting code blocks): add clarification; relevant for blocks ony. Start with a capitalized letter: "Add clarification." Don't use semi-column. "Clarify" is enough IMO. > * doc/org.texi (exports): add clarification; relevant for blocks only. When two changes get the same description use this: * doc/org.texi (Exporting code blocks, exports): Clarify. In general, you can make it easier for you by reviewing the patch (hit `C-x v =' in the modified org.texi buffer), and then hit `C-x 4 a' on each change to create a Changelog buffer that you can safely edit. > The documentation change resulted from the expectation that > `:exports' could be used with inline code blocks, which is not the > case. Clarification was added to avoid future confusion on this > Babel block specific header argument. Please use auto-fill and fill-column to something between 72 and 80. > TINYCHANGE Thanks again for the patch, HTH, -- Bastien