unofficial mirror of guix-devel@gnu.org 
 help / color / mirror / code / Atom feed
* Cleaning up make clean's behavior
@ 2018-06-02 16:23 Jeremiah
  2018-06-02 16:46 ` Nils Gillmann
                   ` (2 more replies)
  0 siblings, 3 replies; 7+ messages in thread
From: Jeremiah @ 2018-06-02 16:23 UTC (permalink / raw)
  To: guix-devel

As running make clean breaks the bootstrap script.
I propose we leverage git's shallow clones (git clone --depth 1 $URL)
and include the .git directory with the repo such that we could simply
have make clean check for git and if it exists run git clean -xdf and
then only if git fails to exist, fallback to the existing broken form;
which needs to be corrected.

per discussion with g_bor[m] about the default automake clean rules
being used currently; and per their suggestion bringing this question to
this distribution list for further discussion.

Additional wouldn't one want to pack the .git in the tarball to enable a
simplified update method.

jlicht pointed out that this would not be a problem yet for guix, but it
 does seem unconventional. It would not make sense for some bigger-repo projects (e.g. emacs) for sure though

Given that discussion background does anyone have any problems, concerns
or issues with the change proposed?

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

* Re: Cleaning up make clean's behavior
  2018-06-02 16:23 Cleaning up make clean's behavior Jeremiah
@ 2018-06-02 16:46 ` Nils Gillmann
  2018-06-02 16:54   ` Gábor Boskovits
  2018-06-03  0:29 ` Mark H Weaver
  2018-06-04 11:36 ` Ludovic Courtès
  2 siblings, 1 reply; 7+ messages in thread
From: Nils Gillmann @ 2018-06-02 16:46 UTC (permalink / raw)
  To: Jeremiah; +Cc: guix-devel

Hi Jeremiah,

Jeremiah@pdp10.guru transcribed 946 bytes:
> As running make clean breaks the bootstrap script.
> I propose we leverage git's shallow clones (git clone --depth 1 $URL)
> and include the .git directory with the repo such that we could simply
> have make clean check for git and if it exists run git clean -xdf and
> then only if git fails to exist, fallback to the existing broken form;
> which needs to be corrected.
> 
> per discussion with g_bor[m] about the default automake clean rules
> being used currently; and per their suggestion bringing this question to
> this distribution list for further discussion.

Where did this discussion take place? Would be good to read some
details if possible.

ng0

> Additional wouldn't one want to pack the .git in the tarball to enable a
> simplified update method.
> 
> jlicht pointed out that this would not be a problem yet for guix, but it
>  does seem unconventional. It would not make sense for some bigger-repo projects (e.g. emacs) for sure though
> 
> Given that discussion background does anyone have any problems, concerns
> or issues with the change proposed?
> 

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

* Re: Cleaning up make clean's behavior
  2018-06-02 16:46 ` Nils Gillmann
@ 2018-06-02 16:54   ` Gábor Boskovits
  0 siblings, 0 replies; 7+ messages in thread
From: Gábor Boskovits @ 2018-06-02 16:54 UTC (permalink / raw)
  To: Jeremiah, Guix-devel

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

2018-06-02 18:46 GMT+02:00 Nils Gillmann <ng0@n0.is>:

> Hi Jeremiah,
>
> Jeremiah@pdp10.guru transcribed 946 bytes:
> > As running make clean breaks the bootstrap script.
> > I propose we leverage git's shallow clones (git clone --depth 1 $URL)
> > and include the .git directory with the repo such that we could simply
> > have make clean check for git and if it exists run git clean -xdf and
> > then only if git fails to exist, fallback to the existing broken form;
> > which needs to be corrected.
> >
> > per discussion with g_bor[m] about the default automake clean rules
> > being used currently; and per their suggestion bringing this question to
> > this distribution list for further discussion.
>
> Where did this discussion take place? Would be good to read some
> details if possible.
>
> ng0
>
> > Additional wouldn't one want to pack the .git in the tarball to enable a
> > simplified update method.
> >
> > jlicht pointed out that this would not be a problem yet for guix, but it
> >  does seem unconventional. It would not make sense for some bigger-repo
> projects (e.g. emacs) for sure though
> >
> > Given that discussion background does anyone have any problems, concerns
> > or issues with the change proposed?
> >
>
> It was on #guix. Currently I have found what causes the problem with make
clean. I'm working on an alternative fix, but the idea outlined here seems
to worth discussing.

[-- Attachment #2: Type: text/html, Size: 1929 bytes --]

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

* Re: Cleaning up make clean's behavior
  2018-06-02 16:23 Cleaning up make clean's behavior Jeremiah
  2018-06-02 16:46 ` Nils Gillmann
@ 2018-06-03  0:29 ` Mark H Weaver
  2018-06-03  6:29   ` Vincent Legoll
  2018-06-04 11:36 ` Ludovic Courtès
  2 siblings, 1 reply; 7+ messages in thread
From: Mark H Weaver @ 2018-06-03  0:29 UTC (permalink / raw)
  To: Jeremiah; +Cc: guix-devel

Jeremiah@pdp10.guru writes:

> As running make clean breaks the bootstrap script.
> I propose we leverage git's shallow clones (git clone --depth 1 $URL)
> and include the .git directory with the repo such that we could simply
> have make clean check for git and if it exists run git clean -xdf and
> then only if git fails to exist, fallback to the existing broken form;
> which needs to be corrected.

I don't think we should change "make clean" to delete files in the
working directory that were not created by our build system.

I, for one, keep many files of notes and draft patches in my Guix
working directory.  I've been doing this for years.  If "make clean"
were suddenly changed to delete all of these files, I would lose data
and probably be very angry.

I happened to see this message, but if I had been focused on other
things, I might well have missed it.  It's not practical for us to
ensure that everyone is warned about this proposed change, so there's no
way for us to avoid the risk that valuable user data would be deleted.

Therefore, we should reject this proposal.

In general, we should be *very* reluctant to delete files that we know
nothing about.  We should be reluctant to do it even if there were a
very compelling reason for it.  Simply wanting to clean up to avoid
unsightly clutter falls far short of a compelling reason.

Thoughts?

      Mark

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

* Re: Cleaning up make clean's behavior
  2018-06-03  0:29 ` Mark H Weaver
@ 2018-06-03  6:29   ` Vincent Legoll
  0 siblings, 0 replies; 7+ messages in thread
From: Vincent Legoll @ 2018-06-03  6:29 UTC (permalink / raw)
  To: Mark H Weaver; +Cc: guix-devel@gnu.org, Jeremiah@pdp10.guru

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

Hello


On Sunday, June 3, 2018, Mark H Weaver <mhw@netris.org> wrote:

> I don't think we should change "make clean" to delete files in the
> working directory that were not created by our build system.
>
> I, for one, keep many files of notes and draft patches in my Guix
> working directory.  I've been doing this for years.  If "make clean"
> were suddenly changed to delete all of these files, I would lose data
> and probably be very angry.
>
> I happened to see this message, but if I had been focused on other
> things, I might well have missed it.  It's not practical for us to
> ensure that everyone is warned about this proposed change, so there's no
> way for us to avoid the risk that valuable user data would be deleted.
>
> Therefore, we should reject this proposal.
>
> In general, we should be *very* reluctant to delete files that we know
> nothing about.  We should be reluctant to do it even if there were a
> very compelling reason for it.  Simply wanting to clean up to avoid
> unsightly clutter falls far short of a compelling reason.
>
> Thoughts?
>
>       Mark
>
>
+1 totally agree


-- 
Vincent Legoll

[-- Attachment #2: Type: text/html, Size: 1464 bytes --]

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

* Re: Cleaning up make clean's behavior
  2018-06-02 16:23 Cleaning up make clean's behavior Jeremiah
  2018-06-02 16:46 ` Nils Gillmann
  2018-06-03  0:29 ` Mark H Weaver
@ 2018-06-04 11:36 ` Ludovic Courtès
  2018-06-10  6:41   ` Chris Marusich
  2 siblings, 1 reply; 7+ messages in thread
From: Ludovic Courtès @ 2018-06-04 11:36 UTC (permalink / raw)
  To: Jeremiah; +Cc: guix-devel

Hello,

Jeremiah@pdp10.guru skribis:

> As running make clean breaks the bootstrap script.

Could you clarify what the problem is?

(Besides I agree with Mark that “make clean” should not delete files it
doesn’t know about.)

Ludo’.

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

* Re: Cleaning up make clean's behavior
  2018-06-04 11:36 ` Ludovic Courtès
@ 2018-06-10  6:41   ` Chris Marusich
  0 siblings, 0 replies; 7+ messages in thread
From: Chris Marusich @ 2018-06-10  6:41 UTC (permalink / raw)
  To: Ludovic Courtès; +Cc: guix-devel, Jeremiah

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

ludo@gnu.org (Ludovic Courtès) writes:

> Could you clarify what the problem is?
>
> (Besides I agree with Mark that “make clean” should not delete files it
> doesn’t know about.)

I also agree.

FYI, to delete everything and return to a pristine Git repository state,
I usually do this:

    # Clean everything the Makefile knows about.
    make distclean
    # Clean everything Git doesn't track.
    git clean -dfx
    # Reset anything else back to whatever is in HEAD.
    git reset --hard

Be careful though, because doing that is, of course, destructive.

-- 
Chris

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

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

end of thread, other threads:[~2018-06-10  6:41 UTC | newest]

Thread overview: 7+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2018-06-02 16:23 Cleaning up make clean's behavior Jeremiah
2018-06-02 16:46 ` Nils Gillmann
2018-06-02 16:54   ` Gábor Boskovits
2018-06-03  0:29 ` Mark H Weaver
2018-06-03  6:29   ` Vincent Legoll
2018-06-04 11:36 ` Ludovic Courtès
2018-06-10  6:41   ` Chris Marusich

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