all messages for Guix-related lists mirrored at yhetil.org
 help / color / mirror / code / Atom feed
* [bug#57549] [PATCH] gnu: Add comments to the bottom of a few package modules.
@ 2022-09-02 19:01 Christopher Baines
  2022-09-02 19:05 ` Maxime Devos
  2022-09-08 14:20 ` Ludovic Courtès
  0 siblings, 2 replies; 4+ messages in thread
From: Christopher Baines @ 2022-09-02 19:01 UTC (permalink / raw)
  To: 57549

This will hopefully discourage patches which add new packages to the bottom of
files, as this increases the likelyhood of conflicts when applying patches.
---
 gnu/packages/python-science.scm | 5 +++++
 gnu/packages/python-xyz.scm     | 5 +++++
 gnu/packages/ruby.scm           | 5 +++++
 gnu/packages/tex.scm            | 5 +++++
 4 files changed, 20 insertions(+)

diff --git a/gnu/packages/python-science.scm b/gnu/packages/python-science.scm
index 95f60aae7b..1c165aa1de 100644
--- a/gnu/packages/python-science.scm
+++ b/gnu/packages/python-science.scm
@@ -1478,3 +1478,8 @@ (define-public python-opt-einsum
 well as potentially any library which conforms to a standard API. See the
 documentation for more information.")
     (license license:expat)))
+
+;;;
+;;; Avoid adding new packages to the end of this file. Place them above by
+;;; existing packages with similar functionality or similar names.
+;;;
diff --git a/gnu/packages/python-xyz.scm b/gnu/packages/python-xyz.scm
index b058704172..e0e144de29 100644
--- a/gnu/packages/python-xyz.scm
+++ b/gnu/packages/python-xyz.scm
@@ -30738,3 +30738,8 @@ (define-public python-lief
      "@code{python-lief} is a cross platform library which can parse, modify
 and abstract ELF, PE and MachO formats.")
     (license license:asl2.0)))
+
+;;;
+;;; Avoid adding new packages to the end of this file. Place them above by
+;;; existing packages with similar functionality or similar names.
+;;;
diff --git a/gnu/packages/ruby.scm b/gnu/packages/ruby.scm
index 94c3bfeb6f..a0c3990b98 100644
--- a/gnu/packages/ruby.scm
+++ b/gnu/packages/ruby.scm
@@ -13766,3 +13766,8 @@ (define (run-with-output-file file command . args)
 has not yet been packaged for Guix.")
     (license license:bsd-2)
     (properties `((upstream-name . "anystyle-cli")))))
+
+;;;
+;;; Avoid adding new packages to the end of this file. Place them above by
+;;; existing packages with similar functionality or similar names.
+;;;
diff --git a/gnu/packages/tex.scm b/gnu/packages/tex.scm
index 212c7eb4a9..c09bae4303 100644
--- a/gnu/packages/tex.scm
+++ b/gnu/packages/tex.scm
@@ -11572,3 +11572,8 @@ (define-public texlive-sourcecodepro
 Adobe in both Adobe Type 1 and OpenType formats, plus macros supporting the
 use of the fonts in LaTeX (Type 1) and XeLaTeX/LuaLaTeX (OTF).")
     (license (list license:lppl1.3+ license:silofl1.1))))
+
+;;;
+;;; Avoid adding new packages to the end of this file. Place them above by
+;;; existing packages with similar functionality or similar names.
+;;;
-- 
2.37.0





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

* [bug#57549] [PATCH] gnu: Add comments to the bottom of a few package modules.
  2022-09-02 19:01 [bug#57549] [PATCH] gnu: Add comments to the bottom of a few package modules Christopher Baines
@ 2022-09-02 19:05 ` Maxime Devos
  2022-09-08 14:20 ` Ludovic Courtès
  1 sibling, 0 replies; 4+ messages in thread
From: Maxime Devos @ 2022-09-02 19:05 UTC (permalink / raw)
  To: Christopher Baines, 57549


[-- Attachment #1.1.1: Type: text/plain, Size: 550 bytes --]


On 02-09-2022 21:01, Christopher Baines wrote:
> This will hopefully discourage patches which add new packages to the bottom of
> files, as this increases the likelyhood of conflicts when applying patches.
> ---
>   gnu/packages/python-science.scm | 5 +++++
>   gnu/packages/python-xyz.scm     | 5 +++++
>   gnu/packages/ruby.scm           | 5 +++++
>   gnu/packages/tex.scm            | 5 +++++
>   4 files changed, 20 insertions(+)

crates-io.scm could benefit from such lines as well.

Sounds useful, LGTM.

Greetings,
Maxime.


[-- Attachment #1.1.2: OpenPGP public key --]
[-- Type: application/pgp-keys, Size: 929 bytes --]

[-- Attachment #2: OpenPGP digital signature --]
[-- Type: application/pgp-signature, Size: 236 bytes --]

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

* [bug#57549] [PATCH] gnu: Add comments to the bottom of a few package modules.
  2022-09-02 19:01 [bug#57549] [PATCH] gnu: Add comments to the bottom of a few package modules Christopher Baines
  2022-09-02 19:05 ` Maxime Devos
@ 2022-09-08 14:20 ` Ludovic Courtès
  2022-09-10 10:43   ` bug#57549: " Christopher Baines
  1 sibling, 1 reply; 4+ messages in thread
From: Ludovic Courtès @ 2022-09-08 14:20 UTC (permalink / raw)
  To: Christopher Baines; +Cc: 57549

Hi,

Christopher Baines <mail@cbaines.net> skribis:

> This will hopefully discourage patches which add new packages to the bottom of
> files, as this increases the likelyhood of conflicts when applying patches.

Please mention the modified files here.

> +;;;
> +;;; Avoid adding new packages to the end of this file. Place them above by

Maybe start with “To reduce the chances of a merge conflict, avoid […]”?

Anyway, LGTM!

Ludo’.




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

* bug#57549: [PATCH] gnu: Add comments to the bottom of a few package modules.
  2022-09-08 14:20 ` Ludovic Courtès
@ 2022-09-10 10:43   ` Christopher Baines
  0 siblings, 0 replies; 4+ messages in thread
From: Christopher Baines @ 2022-09-10 10:43 UTC (permalink / raw)
  To: 57549-done

[-- Attachment #1: Type: text/plain, Size: 746 bytes --]


Ludovic Courtès <ludo@gnu.org> writes:

> Christopher Baines <mail@cbaines.net> skribis:
>
>> This will hopefully discourage patches which add new packages to the bottom of
>> files, as this increases the likelyhood of conflicts when applying patches.
>
> Please mention the modified files here.

Done.

>> +;;;
>> +;;; Avoid adding new packages to the end of this file. Place them above by
>
> Maybe start with “To reduce the chances of a merge conflict, avoid […]”?

I've added this in the second sentence, since I want to keep the
instruction first as this might make it harder to skip over and ignore.

> Anyway, LGTM!

Thanks Ludo and Maxime, pushed this as
fe42befd64df016d755f56107a407f5e4b5c3814.

Chris

[-- Attachment #2: signature.asc --]
[-- Type: application/pgp-signature, Size: 987 bytes --]

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

end of thread, other threads:[~2022-09-10 10:46 UTC | newest]

Thread overview: 4+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2022-09-02 19:01 [bug#57549] [PATCH] gnu: Add comments to the bottom of a few package modules Christopher Baines
2022-09-02 19:05 ` Maxime Devos
2022-09-08 14:20 ` Ludovic Courtès
2022-09-10 10:43   ` bug#57549: " Christopher Baines

Code repositories for project(s) associated with this external index

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

This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.