unofficial mirror of guix-devel@gnu.org 
 help / color / mirror / Atom feed
* Re: branch core-updates updated: packages: Fix 'base32' used before definition warning.
       [not found] <20210110222441.10557.47179@vcs0.savannah.gnu.org>
@ 2021-01-11  8:49 ` Ludovic Courtès
  2021-01-11 13:16   ` Maxim Cournoyer
  0 siblings, 1 reply; 2+ messages in thread
From: Ludovic Courtès @ 2021-01-11  8:49 UTC (permalink / raw)
  To: guix-devel; +Cc: Maxim Cournoyer

Hi Maxim,

guix-commits@gnu.org skribis:

> commit 31ebecf73dadb51ccc72053f54dee36c93a2d0ab
> Author: Maxim Cournoyer <maxim.cournoyer@gmail.com>
> AuthorDate: Sun Jan 10 08:42:42 2021 -0500
>
>     packages: Fix 'base32' used before definition warning.
>     
>     Before this change, compiling the module would emit the following message:
>     "guix/packages.scm:213:25: warning: macro `base32' used before definition".
>     
>     * guix/packages.scm (define-compile-time-decoder)
>     (base32, base64): Move definitions to the top of the module.

To avoid merge conflicts down the road, could you apply it to ‘master’
as well?  (It’s fine to have it on ‘master’ as it doesn’t trigger any
rebuild.)

Thanks,
Ludo’.


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

* Re: branch core-updates updated: packages: Fix 'base32' used before definition warning.
  2021-01-11  8:49 ` branch core-updates updated: packages: Fix 'base32' used before definition warning Ludovic Courtès
@ 2021-01-11 13:16   ` Maxim Cournoyer
  0 siblings, 0 replies; 2+ messages in thread
From: Maxim Cournoyer @ 2021-01-11 13:16 UTC (permalink / raw)
  To: Ludovic Courtès; +Cc: guix-devel

Hi Ludovic!

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

> Hi Maxim,
>
> guix-commits@gnu.org skribis:
>
>> commit 31ebecf73dadb51ccc72053f54dee36c93a2d0ab
>> Author: Maxim Cournoyer <maxim.cournoyer@gmail.com>
>> AuthorDate: Sun Jan 10 08:42:42 2021 -0500
>>
>>     packages: Fix 'base32' used before definition warning.
>>
>>     Before this change, compiling the module would emit the following message:
>>     "guix/packages.scm:213:25: warning: macro `base32' used before definition".
>>
>>     * guix/packages.scm (define-compile-time-decoder)
>>     (base32, base64): Move definitions to the top of the module.
>
> To avoid merge conflicts down the road, could you apply it to ‘master’
> as well?  (It’s fine to have it on ‘master’ as it doesn’t trigger any
> rebuild.)

Thanks for letting me know; I was under the impression that the
guix/packages.scm module as a whole would be used in the computation of
any package's derivation hash.  I'm happy to learn it's actually smarter
than this :-).

Cherry-picked to master with commit
8152fd1af551f4bcec2ef59243264c10ff48daea.

Thank you,

Maxim


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

end of thread, other threads:[~2021-01-11 13:17 UTC | newest]

Thread overview: 2+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
     [not found] <20210110222441.10557.47179@vcs0.savannah.gnu.org>
2021-01-11  8:49 ` branch core-updates updated: packages: Fix 'base32' used before definition warning Ludovic Courtès
2021-01-11 13:16   ` Maxim Cournoyer

unofficial mirror of guix-devel@gnu.org 

This inbox may be cloned and mirrored by anyone:

	git clone --mirror https://yhetil.org/guix-devel/0 guix-devel/git/0.git

	# If you have public-inbox 1.1+ installed, you may
	# initialize and index your mirror using the following commands:
	public-inbox-init -V2 guix-devel guix-devel/ https://yhetil.org/guix-devel \
		guix-devel@gnu.org
	public-inbox-index guix-devel

Example config snippet for mirrors.
Newsgroups are available over NNTP:
	nntp://news.yhetil.org/yhetil.gnu.guix.devel
	nntp://news.gmane.io/gmane.comp.gnu.guix.devel


AGPL code for this site: git clone http://ou63pmih66umazou.onion/public-inbox.git