Re: 07/07: guix: python-build-system: Fix an outdated comment.

Re: 07/07: guix: python-build-system: Fix an outdated comment.

[Mark H Weaver]

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

Re: 07/07: guix: python-build-system: Fix an outdated comment.

[Leo Famulari]

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.

Re: 07/07: guix: python-build-system: Fix an outdated comment.

[Danny Milosavljevic]

> 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?

Re: 07/07: guix: python-build-system: Fix an outdated comment.

[Ludovic Courtès]

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

Re: 07/07: guix: python-build-system: Fix an outdated comment.

[Hartmut Goebel]

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 |