unofficial mirror of guix-devel@gnu.org 
 help / color / mirror / code / Atom feed
* Re: GNU Guixguix source archive branch, master, updated. v0.3-85-gda7cabd
       [not found] <E1VEm2A-0007ez-Of@vcs.savannah.gnu.org>
@ 2013-08-28 21:17 ` Ludovic Courtès
  2013-08-28 21:52   ` Andreas Enge
  0 siblings, 1 reply; 4+ messages in thread
From: Ludovic Courtès @ 2013-08-28 21:17 UTC (permalink / raw)
  To: Andreas Enge; +Cc: guix-devel

Andreas Enge <andreas@enge.fr> skribis:

> commit da7cabd46b20708d083329bd7ec9fd36bf218895
> Author: Andreas Enge <andreas@enge.fr>
> Date:   Wed Aug 28 22:04:52 2013 +0200
>
>     doc: Shuffle some text around.
>     
>     * doc/guix.texi: Drop duplicate copyright notice, start section
>         "Packaging Guidelines" with existant text.

I’d prefer discussing non-trivial doc changes on the list.

(Am I in rant mode or something?  ;-))

Regarding the patch: I think the duplicate copyright notice is needed so
that it appears in the Info output.  Could you check that?

About the structure: I thought an “Adding New Packages” section that
would include both the licensing requirements and technical advice made
sense, on the grounds that contributors-to-be need to see all of that.

Rules as suggested by Cyril could go under “Adding New Packages” (and
not “Packaging Guidelines”, as I initially wrote), in appropriate
sub-sections.

WDYT?

Thanks,
Ludo’.

^ permalink raw reply	[flat|nested] 4+ messages in thread

* Re: GNU Guixguix source archive branch, master, updated. v0.3-85-gda7cabd
  2013-08-28 21:17 ` GNU Guixguix source archive branch, master, updated. v0.3-85-gda7cabd Ludovic Courtès
@ 2013-08-28 21:52   ` Andreas Enge
  2013-08-29 20:17     ` Ludovic Courtès
  2013-08-29 20:41     ` Andreas Enge
  0 siblings, 2 replies; 4+ messages in thread
From: Andreas Enge @ 2013-08-28 21:52 UTC (permalink / raw)
  To: Ludovic Courtès; +Cc: guix-devel

On Wed, Aug 28, 2013 at 11:17:18PM +0200, Ludovic Courtès wrote:
> I’d prefer discussing non-trivial doc changes on the list.

I considered them trivial, since the information has not been altered,
just slightly reorganised.

> Regarding the patch: I think the duplicate copyright notice is needed so
> that it appears in the Info output.  Could you check that?

Indeed, so I should add it again.

> About the structure: I thought an “Adding New Packages” section that
> would include both the licensing requirements and technical advice made
> sense, on the grounds that contributors-to-be need to see all of that.
> 
> Rules as suggested by Cyril could go under “Adding New Packages” (and
> not “Packaging Guidelines”, as I initially wrote), in appropriate
> sub-sections.

I find the title "Packaging Guidelines" snappier. And before, there was
a subsection "Packaging Guidelines" inside the section "Adding New Packages",
so we would have ended up with subsubsections containing the different
items. Now it is one level flatter. And still everything that was in
"6.3 Adding New Packages" is in "6.3 Packaging Guidelines", so not much
changed really. The license requirements are in "6.3.1 Software Freedom",
the technical points should become 6.3.2 and so on.

Andreas

^ permalink raw reply	[flat|nested] 4+ messages in thread

* Re: GNU Guixguix source archive branch, master, updated. v0.3-85-gda7cabd
  2013-08-28 21:52   ` Andreas Enge
@ 2013-08-29 20:17     ` Ludovic Courtès
  2013-08-29 20:41     ` Andreas Enge
  1 sibling, 0 replies; 4+ messages in thread
From: Ludovic Courtès @ 2013-08-29 20:17 UTC (permalink / raw)
  To: Andreas Enge; +Cc: guix-devel

Andreas Enge <andreas@enge.fr> skribis:

> On Wed, Aug 28, 2013 at 11:17:18PM +0200, Ludovic Courtès wrote:
>> I’d prefer discussing non-trivial doc changes on the list.
>
> I considered them trivial, since the information has not been altered,
> just slightly reorganised.

In a way, yes it’s trivial.  OTOH, organizing a manual so that it all
fits together can be non-trivial.  So no worries here, though I prefer
if we follow the rules under ‘Commit Access’ in HACKING when in doubt.

>> Regarding the patch: I think the duplicate copyright notice is needed so
>> that it appears in the Info output.  Could you check that?
>
> Indeed, so I should add it again.
>
>> About the structure: I thought an “Adding New Packages” section that
>> would include both the licensing requirements and technical advice made
>> sense, on the grounds that contributors-to-be need to see all of that.
>> 
>> Rules as suggested by Cyril could go under “Adding New Packages” (and
>> not “Packaging Guidelines”, as I initially wrote), in appropriate
>> sub-sections.
>
> I find the title "Packaging Guidelines" snappier. And before, there was
> a subsection "Packaging Guidelines" inside the section "Adding New Packages",
> so we would have ended up with subsubsections containing the different
> items. Now it is one level flatter. And still everything that was in
> "6.3 Adding New Packages" is in "6.3 Packaging Guidelines", so not much
> changed really. The license requirements are in "6.3.1 Software Freedom",
> the technical points should become 6.3.2 and so on.

OK, I see.

I just checked the current “Packaging Guidelines” section, and I feel
things aren’t completely in place:

The first paragraph (“The GNU distribution is nascent”...) is OK as an
intro, but the following paragraphs (about writing package definitions,
using ‘guix build’, etc.) now seem in the wrong place, since it does not
look like a guideline.

Not sure what to do, though.

Ideas?  :-)

Thanks,
Ludo’.

^ permalink raw reply	[flat|nested] 4+ messages in thread

* Re: GNU Guixguix source archive branch, master, updated. v0.3-85-gda7cabd
  2013-08-28 21:52   ` Andreas Enge
  2013-08-29 20:17     ` Ludovic Courtès
@ 2013-08-29 20:41     ` Andreas Enge
  1 sibling, 0 replies; 4+ messages in thread
From: Andreas Enge @ 2013-08-29 20:41 UTC (permalink / raw)
  To: Ludovic Courtès; +Cc: guix-devel

On Wed, Aug 28, 2013 at 11:52:28PM +0200, Andreas Enge wrote:
> On Wed, Aug 28, 2013 at 11:17:18PM +0200, Ludovic Courtès wrote:
> > Regarding the patch: I think the duplicate copyright notice is needed so
> > that it appears in the Info output.  Could you check that?
> Indeed, so I should add it again.

Done in commit 46cb9da2. I enclosed it into @ifinfo - @end ifinfo, so it
appears only in the info output; the dvi, ps and pdf outputs already
contain the first copyright notice.

Andreas

^ permalink raw reply	[flat|nested] 4+ messages in thread

end of thread, other threads:[~2013-08-29 20:41 UTC | newest]

Thread overview: 4+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
     [not found] <E1VEm2A-0007ez-Of@vcs.savannah.gnu.org>
2013-08-28 21:17 ` GNU Guixguix source archive branch, master, updated. v0.3-85-gda7cabd Ludovic Courtès
2013-08-28 21:52   ` Andreas Enge
2013-08-29 20:17     ` Ludovic Courtès
2013-08-29 20:41     ` Andreas Enge

Code repositories for project(s) associated with this public inbox

	https://git.savannah.gnu.org/cgit/guix.git

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for read-only IMAP folder(s) and NNTP newsgroup(s).