Cosmetical change: remove inconsistent "$file ends here"?

unofficial mirror of guix-devel@gnu.org 
 help / color / mirror / code / Atom feed

* Cosmetical change: remove inconsistent "$file ends here"?
@ 2016-09-24 20:00 ng0
  2016-09-25  8:30 ` Alex Kost
  0 siblings, 1 reply; 12+ messages in thread
From: ng0 @ 2016-09-24 20:00 UTC (permalink / raw)
  To: guix-devel

We should either be consistent with this in all files or remove this
altogether in my opinion.

> ng0@shadowwalker ~/src/guix/guix-no-changes$ egrep -nr "ends here"
> gnu/build/vm.scm:323:;;; vm.scm ends here
> gnu/build/file-systems.scm:490:;;; file-systems.scm ends here
> gnu/build/linux-boot.scm:430:;;; linux-initrd.scm ends here
> gnu/build/marionette.scm:206:;;; marionette.scm ends here
> gnu/build/activation.scm:413:;;; activation.scm ends here
> gnu/build/install.scm:224:;;; install.scm ends here
> gnu/build/linux-modules.scm:214:;;; linux-modules.scm ends here
> gnu/build/linux-initrd.scm:144:;;; linux-initrd.scm ends here
> gnu/tests.scm:241:;;; tests.scm ends here
> gnu/services.scm:615:;;; services.scm ends here.
> gnu/system/vm.scm:506:;;; vm.scm ends here
> gnu/system/file-systems.scm:322:;;; file-systems.scm ends here
> gnu/system/nss.scm:236:;;; nss.scm ends here
> gnu/system/pam.scm:319:;;; linux.scm ends here
> gnu/system/mapped-devices.scm:167:;;; mapped-devices.scm ends here
> gnu/system/install.scm:409:;;; install.scm ends here
> gnu/system/grub.scm:303:;;; grub.scm ends here
> gnu/system/locale.scm:222:;;; locale.scm ends here
> gnu/system/shadow.scm:339:;;; shadow.scm ends here
> gnu/system/linux-initrd.scm:257:;;; linux-initrd.scm ends here
> gnu/artwork.scm:40:;;; artwork.scm ends here
> gnu/tests/install.scm:304:;;; install.scm ends here
> gnu/packages/lynx.scm:87:;;; lynx.scm ends here
> gnu/packages/openbox.scm:70:;;; openbox.scm ends here
> gnu/packages/bootstrap.scm:516:;;; bootstrap.scm ends here
> gnu/packages/base.scm:1006:;;; base.scm ends here
> gnu/packages/messaging.scm:787:;;; messaging.scm ends here
> gnu/packages/ld-wrapper.in:240:;;; ld-wrapper.scm ends here
> gnu/packages/make-bootstrap.scm:691:;;; make-bootstrap.scm ends here
> gnu/packages/guile.scm:1268:;;; guile.scm ends here
> gnu/packages/haskell.scm:7424:;;; haskell.scm ends here
> gnu/packages/libedit.scm:52:;;; libedit.scm ends here
> gnu/packages/search.scm:239:;;; search.scm ends here
> gnu/packages/commencement.scm:895:;;; commencement.scm ends here
> gnu/packages/lxde.scm:217:;;; lxde.scm ends here
> gnu/system.scm:813:;;; system.scm ends here
> gnu/services/base.scm:1546:;;; base.scm ends here
> gnu/services/lirc.scm:102:;;; lirc.scm ends here
> gnu/services/herd.scm:228:;;; herd.scm ends here
> gnu/services/shepherd.scm:346:;;; shepherd.scm ends here
> gnu/services/networking.scm:732:;;; networking.scm ends here
> gnu/services/ssh.scm:328:;;; ssh.scm ends here
> gnu/services/avahi.scm:170:;;; avahi.scm ends here
> gnu/services/xorg.scm:462:;;; xorg.scm ends here
> gnu/services/desktop.scm:864:;;; desktop.scm ends here
> gnu/services/mcron.scm:115:;;; mcron.scm ends here
> gnu/services/dbus.scm:221:;;; dbus.scm ends here
> gnu.scm:62:;;; gnu.scm ends here
> build-aux/generate-authors.scm:120:;;; generate-authors.scm ends here
> build-aux/build-self.scm:149:;;; build-self.scm ends here
> build-aux/test-driver.scm:189:;;;; test-driver.scm ends here.
> emacs/guix-about.el:37:;;; guix-about.el ends here
> emacs/guix-read.el:147:;;; guix-read.el ends here
> emacs/guix-ui-license.el:150:;;; guix-ui-license.el ends here
> emacs/guix-command.el:830:;;; guix-command.el ends here
> emacs/guix-devel.el:382:;;; guix-devel.el ends here
> emacs/guix-messages.el:247:;;; guix-messages.el ends here
> emacs/guix-entry.el:59:;;; guix-entry.el ends here
> emacs/guix-config.el.in:44:;;; guix-config.el ends here
> emacs/guix-build-log.el:381:;;; guix-build-log.el ends here
> emacs/guix-hydra-build.el:362:;;; guix-hydra-build.el ends here
> emacs/guix-info.el:482:;;; guix-info.el ends here
> emacs/guix-ui-location.el:83:;;; guix-ui-location.el ends here
> emacs/guix-ui-generation.el:456:;;; guix-ui-generation.el ends here
> emacs/guix-license.el:65:;;; guix-license.el ends here
> emacs/guix-guile.el:98:;;; guix-guile.el ends here
> emacs/guix-hydra.el:367:;;; guix-hydra.el ends here
> emacs/guix-config.el:44:;;; guix-config.el ends here
> emacs/guix-popup.el:48:;;; guix-popup.el ends here
> emacs/guix-ui-system-generation.el:105:;;; guix-ui-system-generation.el ends here
> emacs/guix-help-vars.el:108:;;; guix-help-vars.el ends here
> emacs/guix-profiles.el:77:;;; guix-profiles.el ends here
> emacs/guix-utils.el:609:;;; guix-utils.el ends here
> emacs/guix-location.el:79:;;; guix-location.el ends here
> emacs/guix-hydra-jobset.el:162:;;; guix-hydra-jobset.el ends here
> emacs/guix-pcomplete.el:370:;;; guix-pcomplete.el ends here
> emacs/guix-ui-package.el:1191:;;; guix-ui-package.el ends here
> emacs/guix-list.el:585:;;; guix-list.el ends here
> emacs/guix-geiser.el:126:;;; guix-geiser.el ends here
> emacs/guix-buffer.el:624:;;; guix-buffer.el ends here
> emacs/guix-history.el:92:;;; guix-history.el ends here
> emacs/guix-emacs.el:123:;;; guix-emacs.el ends here
> emacs/guix-prettify.el:210:;;; guix-prettify.el ends here
> emacs/guix-base.el:377:;;; guix-base.el ends here
> emacs/guix-external.el:88:;;; guix-external.el ends here
> emacs/guix-ui.el:323:;;; guix-ui.el ends here
> emacs/guix-backend.el:393:;;; guix-backend.el ends here
> guix/serialization.scm:381:;;; serialization.scm ends here
> guix/download.scm:450:;;; download.scm ends here
> guix/build/download.scm:765:;;; download.scm ends here
> guix/build/cmake-build-system.scm:84:;;; cmake-build-system.scm ends here
> guix/build/union.scm:156:;;; union.scm ends here
> guix/build/cvs.scm:73:;;; cvs.scm ends here
> guix/build/haskell-build-system.scm:284:;;; haskell-build-system.scm ends here
> guix/build/gnu-dist.scm:92:;;; gnu-dist.scm ends here
> guix/build/rpath.scm:59:;;; rpath.scm ends here
> guix/build/store-copy.scm:69:;;; store-copy.scm ends here
> guix/build/python-build-system.scm:156:;;; python-build-system.scm ends here
> guix/build/r-build-system.scm:112:;;; r-build-system.scm ends here
> guix/build/emacs-utils.scm:113:;;; emacs-utils.scm ends here
> guix/build/bournish.scm:247:;;; bournish.scm ends here
> guix/build/svn.scm:56:;;; svn.scm ends here
> guix/build/syscalls.scm:1285:;;; syscalls.scm ends here
> guix/build/profiles.scm:149:;;; profile.scm ends here
> guix/build/ant-build-system.scm:177:;;; ant-build-system.scm ends here
> guix/build/emacs-build-system.scm:196:;;; emacs-build-system.scm ends here
> guix/build/glib-or-gtk-build-system.scm:226:;;; glib-or-gtk-build-system.scm ends here
> guix/build/waf-build-system.scm:83:;;; waf-build-system.scm ends here
> guix/build/perl-build-system.scm:87:;;; perl-build-system.scm ends here
> guix/build/hg.scm:51:;;; hg.scm ends here
> guix/build/git.scm:64:;;; git.scm ends here
> guix/build/graft.scm:289:;;; graft.scm ends here
> guix/build/gremlin.scm:309:;;; gremlin.scm ends here
> guix/build/pull.scm:148:;;; pull.scm ends here
> guix/combinators.scm:116:;;; combinators.scm ends here
> guix/records.scm:379:;;; records.scm ends here
> guix/scripts/challenge.scm:248:;;; challenge.scm ends here
> guix/scripts/substitute.scm:1053:;;; substitute.scm ends here
> guix/scripts/authenticate.scm:128:;;; authenticate.scm ends here
> guix/scripts/offload.scm:751:;;; offload.scm ends here
> guix/scripts/graph.scm:412:;;; graph.scm ends here
> guix/scripts/import/gnu.scm:105:;;; gnu.scm ends here
> guix/scripts/import/elpa.scm:99:;;; elpa.scm ends here
> guix/scripts/system.scm:881:;;; system.scm ends here
> guix/scripts/pull.scm:245:;;; pull.scm ends here
> guix/tests.scm:286:;;; tests.scm ends here
> guix/base32.scm:303:;;; base32.scm ends here
> guix/search-paths.scm:187:;;; search-paths.scm ends here
> guix/nar.scm:276:;;; nar.scm ends here
> guix/modules.scm:155:;;; modules.scm ends here
> guix/cve.scm:301:;;; cve.scm ends here
> guix/ui.scm:1230:;;; ui.scm ends here
> guix/cvs-download.scm:82:;;; cvs-download.scm ends here
> guix/scripts.scm:139:;;; scripts.scm ends here
> guix/hg-download.scm:84:;;; hg-download.scm ends here
> guix/cpio.scm:253:;;; cpio.scm ends here
> guix/gnu-maintenance.scm:585:;;; gnu-maintenance.scm ends here
> guix/build-system/emacs.scm:137:;;; emacs.scm ends here
> guix/build-system/r.scm:153:;;; r.scm ends here
> guix/build-system/perl.scm:145:;;; perl.scm ends here
> guix/build-system/python.scm:233:;;; python.scm ends here
> guix/build-system/cmake.scm:154:;;; cmake.scm ends here
> guix/build-system/ant.scm:157:;;; ant.scm ends here
> guix/build-system/haskell.scm:140:;;; haskell.scm ends here
> guix/build-system/waf.scm:133:;;; waf.scm ends here
> guix/upstream.scm:243:;;; upstream.scm ends here
> guix/config.scm.in:114:;;; config.scm ends here
> guix/ftp-client.scm:313:;;; ftp-client.scm ends here
> guix/pk-crypto.scm:401:;;; pk-crypto.scm ends here
> guix/gcrypt.scm:49:;;; gcrypt.scm ends here
> guix/svn-download.scm:82:;;; svn-download.scm ends here
> guix/graph.scm:213:;;; graph.scm ends here
> guix/config.scm:114:;;; config.scm ends here
> guix/profiles.scm:1077:;;; profiles.scm ends here
> guix/gnupg.scm:184:;;; gnupg.scm ends here
> guix/pki.scm:220:;;; pki.scm ends here
> guix/import/cabal.scm:858:;;; cabal.scm ends here
> guix/import/gnu.scm:127:;;; gnu.scm ends here
> guix/import/hackage.scm:296:;;; cabal.scm ends here
> guix/import/elpa.scm:279:;;; elpa.scm ends here
> guix/import/snix.scm:466:;;; snix.scm ends here
> guix/import/cran.scm:306:;;; cran.scm ends here
> guix/grafts.scm:311:;;; grafts.scm ends here
> guix/monad-repl.scm:93:;;; monad-repl.scm ends here
> guix/gexp.scm:1168:;;; gexp.scm ends here
> guix/http-client.scm:329:;;; http-client.scm ends here
> guix/hash.scm:169:;;; hash.scm ends here
> guix/zlib.scm:245:;;; zlib.scm ends here
> guix/git-download.scm:101:;;; git-download.scm ends here
> guix/sets.scm:116:;;; sets.scm ends here
> guix/licenses.scm:468:;;; licenses.scm ends here
> srfi/srfi-37.scm.in:233:;;; srfi-37.scm ends here

What do you think?
-- 
              ng0

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

* Re: Cosmetical change: remove inconsistent "$file ends here"?
  2016-09-24 20:00 Cosmetical change: remove inconsistent "$file ends here"? ng0
@ 2016-09-25  8:30 ` Alex Kost
  2016-09-25  9:21   ` ng0
                     ` (3 more replies)
  0 siblings, 4 replies; 12+ messages in thread
From: Alex Kost @ 2016-09-25  8:30 UTC (permalink / raw)
  To: ng0; +Cc: guix-devel

ng0 (2016-09-24 20:00 +0000) wrote:

> We should either be consistent with this in all files or remove this
> altogether in my opinion.
>
>> ng0@shadowwalker ~/src/guix/guix-no-changes$ egrep -nr "ends here"
>> gnu/build/vm.scm:323:;;; vm.scm ends here
...
> What do you think?

I don't know what the original purpose of this convention is, it was
probably invented in those ancient times when dinosaurs walked by
streets, but I kinda like these "ends here" things :-)

The only purpose I see in using them: you can be sure that there will
not appear redundant newlines (introduced by untidy commits) in the end
of files.

Anyway, I vote for leaving them and adding the missing ones.

-- 
Alex

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

* Re: Cosmetical change: remove inconsistent "$file ends here"?
  2016-09-25  8:30 ` Alex Kost
@ 2016-09-25  9:21   ` ng0
  2016-09-26  8:32     ` Christopher Allan Webber
  2016-09-25  9:32   ` John Darrington
                     ` (2 subsequent siblings)
  3 siblings, 1 reply; 12+ messages in thread
From: ng0 @ 2016-09-25  9:21 UTC (permalink / raw)
  To: Alex Kost; +Cc: guix-devel

Alex Kost <alezost@gmail.com> writes:

> ng0 (2016-09-24 20:00 +0000) wrote:
>
>> We should either be consistent with this in all files or remove this
>> altogether in my opinion.
>>
>>> ng0@shadowwalker ~/src/guix/guix-no-changes$ egrep -nr "ends here"
>>> gnu/build/vm.scm:323:;;; vm.scm ends here
> ...
>> What do you think?
>
> I don't know what the original purpose of this convention is, it was
> probably invented in those ancient times when dinosaurs walked by
> streets, but I kinda like these "ends here" things :-)
>
> The only purpose I see in using them: you can be sure that there will
> not appear redundant newlines (introduced by untidy commits) in the end
> of files.
>
> Anyway, I vote for leaving them and adding the missing ones.

Okay, with your explanation this is the more reasonable thing to do.

> -- 
> Alex
>

-- 
              ng0

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

* Re: Cosmetical change: remove inconsistent "$file ends here"?
  2016-09-25  9:21   ` ng0
@ 2016-09-26  8:32     ` Christopher Allan Webber
  2016-09-26  8:45       ` Andy Wingo
  0 siblings, 1 reply; 12+ messages in thread
From: Christopher Allan Webber @ 2016-09-26  8:32 UTC (permalink / raw)
  To: ng0; +Cc: guix-devel, Alex Kost

ng0 writes:

> Alex Kost <alezost@gmail.com> writes:
>
>> ng0 (2016-09-24 20:00 +0000) wrote:
>>
>>> We should either be consistent with this in all files or remove this
>>> altogether in my opinion.
>>>
>>>> ng0@shadowwalker ~/src/guix/guix-no-changes$ egrep -nr "ends here"
>>>> gnu/build/vm.scm:323:;;; vm.scm ends here
>> ...
>>> What do you think?
>>
>> I don't know what the original purpose of this convention is, it was
>> probably invented in those ancient times when dinosaurs walked by
>> streets, but I kinda like these "ends here" things :-)
>>
>> The only purpose I see in using them: you can be sure that there will
>> not appear redundant newlines (introduced by untidy commits) in the end
>> of files.
>>
>> Anyway, I vote for leaving them and adding the missing ones.
>
> Okay, with your explanation this is the more reasonable thing to do.

I'm really fine either way, but that doesn't feel like a super
compelling reason to me.  Who cares if there's a redundant newline at
the end of the file?  Sure, it should be swept up, but one redundant
newline costs a lot less than "don't let any newlines after this".
Plus, do we have proof that no newlines will really appear afterwards?

It feels like cruft that nobody knows the reason for, or has a good
reason in this modern age.  I'd rather they just be trimmed.

But I also don't feel like it matters much, so whatever is decided is
decided. :)

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

* Re: Cosmetical change: remove inconsistent "$file ends here"?
  2016-09-26  8:32     ` Christopher Allan Webber
@ 2016-09-26  8:45       ` Andy Wingo
  0 siblings, 0 replies; 12+ messages in thread
From: Andy Wingo @ 2016-09-26  8:45 UTC (permalink / raw)
  To: guix-devel

On Mon 26 Sep 2016 10:32, Christopher Allan Webber <cwebber@dustycloud.org> writes:

> It feels like cruft that nobody knows the reason for, or has a good
> reason in this modern age.  I'd rather they just be trimmed.
>
> But I also don't feel like it matters much, so whatever is decided is
> decided. :)

For what it's worth: +1 from me.  These things have cost in terms of
linting, code review, need for explaining to new people, and visual
ceremony, and they have very little benefit if any.

But, also like Christopher, I don't care strongly :)

Andy

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

* Re: Cosmetical change: remove inconsistent "$file ends here"?
  2016-09-25  8:30 ` Alex Kost
  2016-09-25  9:21   ` ng0
@ 2016-09-25  9:32   ` John Darrington
  2016-09-25  9:38   ` Danny Milosavljevic
  2016-09-28  8:38   ` Ludovic Courtès
  3 siblings, 0 replies; 12+ messages in thread
From: John Darrington @ 2016-09-25  9:32 UTC (permalink / raw)
  To: Alex Kost; +Cc: guix-devel

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

On Sun, Sep 25, 2016 at 11:30:41AM +0300, Alex Kost wrote:
     ng0 (2016-09-24 20:00 +0000) wrote:
     
     > We should either be consistent with this in all files or remove this
     > altogether in my opinion.
     >
     >> ng0@shadowwalker ~/src/guix/guix-no-changes$ egrep -nr "ends here"
     >> gnu/build/vm.scm:323:;;; vm.scm ends here
     ...
     > What do you think?
     
     I don't know what the original purpose of this convention is, it was
     probably invented in those ancient times when dinosaurs walked by
     streets, but I kinda like these "ends here" things :-)
     
     The only purpose I see in using them: you can be sure that there will
     not appear redundant newlines (introduced by untidy commits) in the end
     of files.
     
     Anyway, I vote for leaving them and adding the missing ones.
     

I have a better idea.  Let's remove these verbose messages, and write an API,
which is capable of opening and streaming files , we could call those functions - 
for example  - "fopen" and "fread".  But here comes the clever bit:   We also
provide a function - let's call it "feof".

This approach will have the advantage that we could use this API to write things
called "editors" which might display some user friendly string such as "End of buffer" 
when the end is encountered.

Once these tools are developed, we could inform the community that they no longer
need to put the string "file ends here" at the end of their files.

WDYT ?


-- 
Avoid eavesdropping.  Send strong encrypted email.
PGP Public key ID: 1024D/2DE827B3 
fingerprint = 8797 A26D 0854 2EAB 0285  A290 8A67 719C 2DE8 27B3
See http://sks-keyservers.net or any PGP keyserver for public key.


[-- Attachment #2: Digital signature --]
[-- Type: application/pgp-signature, Size: 181 bytes --]

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

* Re: Cosmetical change: remove inconsistent "$file ends here"?
  2016-09-25  8:30 ` Alex Kost
  2016-09-25  9:21   ` ng0
  2016-09-25  9:32   ` John Darrington
@ 2016-09-25  9:38   ` Danny Milosavljevic
  2016-09-25 17:45     ` Hartmut Goebel
  2016-09-28  8:38   ` Ludovic Courtès
  3 siblings, 1 reply; 12+ messages in thread
From: Danny Milosavljevic @ 2016-09-25  9:38 UTC (permalink / raw)
  To: Alex Kost; +Cc: guix-devel

Hi,

On Sun, 25 Sep 2016 11:30:41 +0300
Alex Kost <alezost@gmail.com> wrote:
> I don't know what the original purpose of this convention is,

That's probably enough reason to remove them.

> it was
> probably invented in those ancient times when dinosaurs walked by
> streets, but I kinda like these "ends here" things :-)
> 
> The only purpose I see in using them: you can be sure that there will
> not appear redundant newlines (introduced by untidy commits) in the end
> of files.

I speculate it's that when you print source code out you know that you are not missing a sheet when you take them with you. Doesn't really serve a purpose now (also, diff tools show you if you forget the newline before the end of file - also many editors do, even nano). 

That said, touching it at all causes churn - not sure what the upside is.

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

* Re: Cosmetical change: remove inconsistent "$file ends here"?
  2016-09-25  9:38   ` Danny Milosavljevic
@ 2016-09-25 17:45     ` Hartmut Goebel
  2016-09-25 17:55       ` Vincent Legoll
  0 siblings, 1 reply; 12+ messages in thread
From: Hartmut Goebel @ 2016-09-25 17:45 UTC (permalink / raw)
  To: guix-devel

Am 25.09.2016 um 11:38 schrieb Danny Milosavljevic:
> That's probably enough reason to remove them.

+1

-- 
Regards
Hartmut Goebel

| Hartmut Goebel          | h.goebel@crazy-compilers.com               |
| www.crazy-compilers.com | compilers which you thought are impossible |

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

* Re: Cosmetical change: remove inconsistent "$file ends here"?
  2016-09-25 17:45     ` Hartmut Goebel
@ 2016-09-25 17:55       ` Vincent Legoll
  2016-09-25 17:59         ` John Darrington
  0 siblings, 1 reply; 12+ messages in thread
From: Vincent Legoll @ 2016-09-25 17:55 UTC (permalink / raw)
  To: Hartmut Goebel; +Cc: guix-devel

On Sun, Sep 25, 2016 at 7:45 PM, Hartmut Goebel
<h.goebel@crazy-compilers.com> wrote:
> Am 25.09.2016 um 11:38 schrieb Danny Milosavljevic:
>> That's probably enough reason to remove them.
>
> +1

+1, but please do not post a patchbomb^Wseries for review on the ML,
just script it
and merge at the most appropriate time wrt rebuilding the world...

-- 
Vincent Legoll

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

* Re: Cosmetical change: remove inconsistent "$file ends here"?
  2016-09-25 17:55       ` Vincent Legoll
@ 2016-09-25 17:59         ` John Darrington
  0 siblings, 0 replies; 12+ messages in thread
From: John Darrington @ 2016-09-25 17:59 UTC (permalink / raw)
  To: Vincent Legoll; +Cc: guix-devel

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

On Sun, Sep 25, 2016 at 07:55:48PM +0200, Vincent Legoll wrote:
     On Sun, Sep 25, 2016 at 7:45 PM, Hartmut Goebel
     <h.goebel@crazy-compilers.com> wrote:
     > Am 25.09.2016 um 11:38 schrieb Danny Milosavljevic:
     >> That's probably enough reason to remove them.
     >
     > +1
     
     +1, but please do not post a patchbomb^Wseries for review on the ML,
     just script it
     and merge at the most appropriate time wrt rebuilding the world...

Such a change won't cause anything to be rebuilt.

J'

-- 
Avoid eavesdropping.  Send strong encrypted email.
PGP Public key ID: 1024D/2DE827B3 
fingerprint = 8797 A26D 0854 2EAB 0285  A290 8A67 719C 2DE8 27B3
See http://sks-keyservers.net or any PGP keyserver for public key.


[-- Attachment #2: Digital signature --]
[-- Type: application/pgp-signature, Size: 181 bytes --]

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

* Re: Cosmetical change: remove inconsistent "$file ends here"?
  2016-09-25  8:30 ` Alex Kost
                     ` (2 preceding siblings ...)
  2016-09-25  9:38   ` Danny Milosavljevic
@ 2016-09-28  8:38   ` Ludovic Courtès
  2016-10-06 19:18     ` ng0
  3 siblings, 1 reply; 12+ messages in thread
From: Ludovic Courtès @ 2016-09-28  8:38 UTC (permalink / raw)
  To: Alex Kost; +Cc: guix-devel

Alex Kost <alezost@gmail.com> skribis:

> ng0 (2016-09-24 20:00 +0000) wrote:
>
>> We should either be consistent with this in all files or remove this
>> altogether in my opinion.
>>
>>> ng0@shadowwalker ~/src/guix/guix-no-changes$ egrep -nr "ends here"
>>> gnu/build/vm.scm:323:;;; vm.scm ends here
> ...
>> What do you think?
>
> I don't know what the original purpose of this convention is, it was
> probably invented in those ancient times when dinosaurs walked by
> streets, but I kinda like these "ends here" things :-)

Yeah, I add them occasionally out of habit, but it’s not useful.

> Anyway, I vote for leaving them and adding the missing ones.

I vote for the status quo.  :-)  There’s not much value in spending time
either way IMO.

Ludo’.

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

* Re: Cosmetical change: remove inconsistent "$file ends here"?
  2016-09-28  8:38   ` Ludovic Courtès
@ 2016-10-06 19:18     ` ng0
  0 siblings, 0 replies; 12+ messages in thread
From: ng0 @ 2016-10-06 19:18 UTC (permalink / raw)
  To: Ludovic Courtès, Alex Kost; +Cc: guix-devel

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

> Alex Kost <alezost@gmail.com> skribis:
>
>> ng0 (2016-09-24 20:00 +0000) wrote:
>>
>>> We should either be consistent with this in all files or remove this
>>> altogether in my opinion.
>>>
>>>> ng0@shadowwalker ~/src/guix/guix-no-changes$ egrep -nr "ends here"
>>>> gnu/build/vm.scm:323:;;; vm.scm ends here
>> ...
>>> What do you think?
>>
>> I don't know what the original purpose of this convention is, it was
>> probably invented in those ancient times when dinosaurs walked by
>> streets, but I kinda like these "ends here" things :-)
>
> Yeah, I add them occasionally out of habit, but it’s not useful.
>
>> Anyway, I vote for leaving them and adding the missing ones.
>
> I vote for the status quo.  :-)  There’s not much value in spending time
> either way IMO.
>
> Ludo’.
>

Okay, let's keep it this way. There's nothing to be gained from this.

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

end of thread, other threads:[~2016-10-06 19:19 UTC | newest]

Thread overview: 12+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2016-09-24 20:00 Cosmetical change: remove inconsistent "$file ends here"? ng0
2016-09-25  8:30 ` Alex Kost
2016-09-25  9:21   ` ng0
2016-09-26  8:32     ` Christopher Allan Webber
2016-09-26  8:45       ` Andy Wingo
2016-09-25  9:32   ` John Darrington
2016-09-25  9:38   ` Danny Milosavljevic
2016-09-25 17:45     ` Hartmut Goebel
2016-09-25 17:55       ` Vincent Legoll
2016-09-25 17:59         ` John Darrington
2016-09-28  8:38   ` Ludovic Courtès
2016-10-06 19:18     ` ng0

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).