unofficial mirror of guix-devel@gnu.org 
 help / color / mirror / code / Atom feed
* Re: 07/07: guix: python-build-system: Fix an outdated comment.
       [not found] ` <20161013152324.EF9C7220170@vcs.savannah.gnu.org>
@ 2016-10-13 19:32   ` Mark H Weaver
  2016-10-13 19:39     ` Leo Famulari
  0 siblings, 1 reply; 5+ messages in thread
From: Mark H Weaver @ 2016-10-13 19:32 UTC (permalink / raw)
  To: Hartmut Goebel; +Cc: guix-devel

h.goebel@crazy-compilers.com (Hartmut Goebel) writes:

> htgoebel pushed a commit to branch master
> in repository guix.
>
> commit 635a7af45d6e2105ad65d1a9531126cc232a2a50
> Author: Hartmut Goebel <h.goebel@crazy-compilers.com>
> Date:   Sun Oct 2 18:48:56 2016 +0200
>
>     guix: python-build-system: Fix an outdated comment.
>     
>     The python-build-system uses phases the build and install, but not
>     configure. So the old comment was plain wrong since Sept. 2013, when the build
>     phase has been added.
> ---
>  guix/build/python-build-system.scm |    3 +--
>  1 file changed, 1 insertion(+), 2 deletions(-)
>
> diff --git a/guix/build/python-build-system.scm b/guix/build/python-build-system.scm
> index 9109fb4..e906e60 100644
> --- a/guix/build/python-build-system.scm
> +++ b/guix/build/python-build-system.scm
> @@ -137,8 +137,7 @@ installed with setuptools."
>      #t))
>  
>  (define %standard-phases
> -  ;; 'configure' and 'build' phases are not needed.  Everything is done during
> -  ;; 'install'.
> +  ;; 'configure' phase is not needed.
>    (modify-phases gnu:%standard-phases
>      (add-after 'unpack 'ensure-no-mtimes-pre-1980 ensure-no-mtimes-pre-1980)
>      (delete 'configure)

This change triggered over 6300 rebuilds on 'master'.  I reverted it.
Please beware that changing some files in guix/build/*.scm can trigger a
large number of rebuilds, because some of these files are implicitly
used as inputs to a large number of builds.

    Regards,
      Mark

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

* Re: 07/07: guix: python-build-system: Fix an outdated comment.
  2016-10-13 19:32   ` 07/07: guix: python-build-system: Fix an outdated comment Mark H Weaver
@ 2016-10-13 19:39     ` Leo Famulari
  2016-10-13 20:45       ` Danny Milosavljevic
  2016-10-14  7:55       ` Hartmut Goebel
  0 siblings, 2 replies; 5+ messages in thread
From: Leo Famulari @ 2016-10-13 19:39 UTC (permalink / raw)
  To: Mark H Weaver; +Cc: guix-devel

On Thu, Oct 13, 2016 at 03:32:09PM -0400, Mark H Weaver wrote:
> h.goebel@crazy-compilers.com (Hartmut Goebel) writes:
> 
> > htgoebel pushed a commit to branch master
> > in repository guix.
> >
> > commit 635a7af45d6e2105ad65d1a9531126cc232a2a50
> > Author: Hartmut Goebel <h.goebel@crazy-compilers.com>
> > Date:   Sun Oct 2 18:48:56 2016 +0200
> >
> >     guix: python-build-system: Fix an outdated comment.
> >     
> >     The python-build-system uses phases the build and install, but not
> >     configure. So the old comment was plain wrong since Sept. 2013, when the build
> >     phase has been added.
>
> This change triggered over 6300 rebuilds on 'master'.  I reverted it.
> Please beware that changing some files in guix/build/*.scm can trigger a
> large number of rebuilds, because some of these files are implicitly
> used as inputs to a large number of builds.

Sorry, I didn't realize that altering comments here would have an
effect; I would have warned Hartmut if I had.

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

* Re: 07/07: guix: python-build-system: Fix an outdated comment.
  2016-10-13 19:39     ` Leo Famulari
@ 2016-10-13 20:45       ` Danny Milosavljevic
  2016-10-15 22:16         ` Ludovic Courtès
  2016-10-14  7:55       ` Hartmut Goebel
  1 sibling, 1 reply; 5+ messages in thread
From: Danny Milosavljevic @ 2016-10-13 20:45 UTC (permalink / raw)
  To: Leo Famulari, guix-devel

> Sorry, I didn't realize that altering comments here would have an
> effect; I would have warned Hartmut if I had.

Out of curiousity, why does it have an effect?

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

* Re: 07/07: guix: python-build-system: Fix an outdated comment.
  2016-10-13 19:39     ` Leo Famulari
  2016-10-13 20:45       ` Danny Milosavljevic
@ 2016-10-14  7:55       ` Hartmut Goebel
  1 sibling, 0 replies; 5+ messages in thread
From: Hartmut Goebel @ 2016-10-14  7:55 UTC (permalink / raw)
  To: Leo Famulari, Mark H Weaver; +Cc: guix-devel

Am 13.10.2016 um 21:39 schrieb Leo Famulari:
>> > This change triggered over 6300 rebuilds on 'master'.  I reverted it.
>> > Please beware that changing some files in guix/build/*.scm can trigger a
>> > large number of rebuilds, because some of these files are implicitly
>> > used as inputs to a large number of builds.
> Sorry, I didn't realize that altering comments here would have an
> effect; I would have warned Hartmut if I had.

I'm sorry, too. I didn't expect that changing a comment would have any
impact on building packages. Aren't the the files converted to some
canonical form prior to deciding whether they changed?

(I'll take this change to the wip-python-build-system).

-- 
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] 5+ messages in thread

* Re: 07/07: guix: python-build-system: Fix an outdated comment.
  2016-10-13 20:45       ` Danny Milosavljevic
@ 2016-10-15 22:16         ` Ludovic Courtès
  0 siblings, 0 replies; 5+ messages in thread
From: Ludovic Courtès @ 2016-10-15 22:16 UTC (permalink / raw)
  To: Danny Milosavljevic; +Cc: guix-devel

Danny Milosavljevic <dannym@scratchpost.org> skribis:

>> Sorry, I didn't realize that altering comments here would have an
>> effect; I would have warned Hartmut if I had.
>
> Out of curiousity, why does it have an effect?

The file is imported in the build environment (chroot) of all the
packages that use ‘python-build-system’.  It’s an input of the
derivation of those packages, so when it changes, all these derivations
change.

Hartmut Goebel <h.goebel@crazy-compilers.com> skribis:

>> Sorry, I didn't realize that altering comments here would have an
>> effect; I would have warned Hartmut if I had.
>
> I'm sorry, too. I didn't expect that changing a comment would have any
> impact on building packages. Aren't the the files converted to some
> canonical form prior to deciding whether they changed?

No.  There’s no “decision” as to whether they changed; instead, they
just happen to be part of the hash of the output file name.

That said, don’t be sorry gentlefolks: these things happen.  ;-)

Ludo’.

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

end of thread, other threads:[~2016-10-15 22:16 UTC | newest]

Thread overview: 5+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
     [not found] <20161013152324.1276.70078@vcs.savannah.gnu.org>
     [not found] ` <20161013152324.EF9C7220170@vcs.savannah.gnu.org>
2016-10-13 19:32   ` 07/07: guix: python-build-system: Fix an outdated comment Mark H Weaver
2016-10-13 19:39     ` Leo Famulari
2016-10-13 20:45       ` Danny Milosavljevic
2016-10-15 22:16         ` Ludovic Courtès
2016-10-14  7:55       ` Hartmut Goebel

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