'package-name-version' and 'strip-store-file-name' does the same thing with different ways

'package-name-version' and 'strip-store-file-name' does the same thing with different ways

[Alex Vong]

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

Hi guix,

Currently, I am attempting to create clojure-build-system.

In the process of doing so, I discover 2 procedures with different names
actually does the same thing with different ways. The 2 procedures
are 'package-name-version' and 'strip-store-file-name'.

'strip-store-file-name' is defined in (guix build utils), while
'package-name-version' is defined in (guix build haskell-build-system)
and (guix java-utils).

I think we should remove the ones not defined in (guix build utils) and
provide 'package-name-version' as alias for 'strip-store-file-name' in
(guix build utils). What do you think?

Cheers,
Alex

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

Re: 'package-name-version' and 'strip-store-file-name' does the same thing with different ways

[Ludovic Courtès]

Hello Alex,

Alex Vong <alexvong1995@gmail.com> skribis:

> I think we should remove the ones not defined in (guix build utils) and
> provide 'package-name-version' as alias for 'strip-store-file-name' in
> (guix build utils). What do you think?

I think we can simply keep ‘strip-store-file-name’, without the alias,
and modify existing code to use it.

(That may trigger a lot of rebuilds though, so we’ll have to see which
branch to apply it to.)

Thanks for noticing!

Ludo’.

Re: 'package-name-version' and 'strip-store-file-name' does the same thing with different ways

[Alex Vong]

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

Hi Ludo,

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

> Hello Alex,
>
> Alex Vong <alexvong1995@gmail.com> skribis:
>
>> I think we should remove the ones not defined in (guix build utils) and
>> provide 'package-name-version' as alias for 'strip-store-file-name' in
>> (guix build utils). What do you think?
>
> I think we can simply keep ‘strip-store-file-name’, without the alias,
> and modify existing code to use it.
>
OK.

> (That may trigger a lot of rebuilds though, so we’ll have to see which
> branch to apply it to.)
>
> Thanks for noticing!
>
You're welcomed!

> Ludo’.

Btw, I also noticed that the procedure 'flatten' are defined in
(gnu services telephony), (gnu services web) and (guix import utils).
[I was about to define my own version of it...]
I like the version using 'fold-right' together with 'match-lambda*' the
most. Should we move it (guix build utils) and remove the rest?

Similarly, 'file-sans-extension' are defined in both (guix utils) and
(guix build guile-build-system). They are both the same. Should we move
them to (guix build utils) as well?

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

Re: 'package-name-version' and 'strip-store-file-name' does the same thing with different ways

[Ludovic Courtès]

Hello Alex,

Alex Vong <alexvong1995@gmail.com> skribis:

> Btw, I also noticed that the procedure 'flatten' are defined in
> (gnu services telephony), (gnu services web) and (guix import utils).
> [I was about to define my own version of it...]
> I like the version using 'fold-right' together with 'match-lambda*' the
> most. Should we move it (guix build utils) and remove the rest?

About ‘flatten’ specifically, I feel like it’s not a good sign in terms
of programming style when we end up using it (though we’d need to study
these specific use cases), so I’m reluctant to “standardizing” it.  :-)

> Similarly, 'file-sans-extension' are defined in both (guix utils) and
> (guix build guile-build-system). They are both the same. Should we move
> them to (guix build utils) as well?

Yes, sounds like a good idea!

Thanks,
Ludo’.

Re: 'package-name-version' and 'strip-store-file-name' does the same thing with different ways

[Alex Vong]

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

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

> Hello Alex,
>
> Alex Vong <alexvong1995@gmail.com> skribis:
>
>> Btw, I also noticed that the procedure 'flatten' are defined in
>> (gnu services telephony), (gnu services web) and (guix import utils).
>> [I was about to define my own version of it...]
>> I like the version using 'fold-right' together with 'match-lambda*' the
>> most. Should we move it (guix build utils) and remove the rest?
>
> About ‘flatten’ specifically, I feel like it’s not a good sign in terms
> of programming style when we end up using it (though we’d need to study
> these specific use cases), so I’m reluctant to “standardizing” it.  :-)
>
Your comment lead me to find out that 'append-map' is actually what I
want. I am curious why it is an indication of bad style.

>> Similarly, 'file-sans-extension' are defined in both (guix utils) and
>> (guix build guile-build-system). They are both the same. Should we move
>> them to (guix build utils) as well?
>
> Yes, sounds like a good idea!
>
OK!

> Thanks,
> Ludo’.

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

Re: 'package-name-version' and 'strip-store-file-name' does the same thing with different ways

[Ricardo Wurmus]

Ludovic Courtès <ludo@gnu.org> writes:
>> Similarly, 'file-sans-extension' are defined in both (guix utils) and
>> (guix build guile-build-system). They are both the same. Should we move
>> them to (guix build utils) as well?
>
> Yes, sounds like a good idea!

‘file-sans-extension’ is used on the host-side, e.g. in scripts/build.scm.
Is it still fine to move it to a build-side module only?

--
Ricardo

Re: 'package-name-version' and 'strip-store-file-name' does the same thing with different ways

[Ludovic Courtès]

Hi Alex,

Alex Vong <alexvong1995@gmail.com> skribis:

> ludo@gnu.org (Ludovic Courtès) writes:
>
>> Hello Alex,
>>
>> Alex Vong <alexvong1995@gmail.com> skribis:
>>
>>> Btw, I also noticed that the procedure 'flatten' are defined in
>>> (gnu services telephony), (gnu services web) and (guix import utils).
>>> [I was about to define my own version of it...]
>>> I like the version using 'fold-right' together with 'match-lambda*' the
>>> most. Should we move it (guix build utils) and remove the rest?
>>
>> About ‘flatten’ specifically, I feel like it’s not a good sign in terms
>> of programming style when we end up using it (though we’d need to study
>> these specific use cases), so I’m reluctant to “standardizing” it.  :-)
>>
> Your comment lead me to find out that 'append-map' is actually what I
> want. I am curious why it is an indication of bad style.

My intuition is that often, when one feels a need for ‘flatten’, they’d
probably be using ‘append’, ‘concatenate’, etc., or perhaps they’re
using the wrong data structure in the first place.  But again, we’d have
to look more closely at the existing use cases to see more concretely to
what extent they could be written differently.

Ludo’.

Re: 'package-name-version' and 'strip-store-file-name' does the same thing with different ways

[Ludovic Courtès]

Ricardo Wurmus <rekado@elephly.net> skribis:

> Ludovic Courtès <ludo@gnu.org> writes:
>>> Similarly, 'file-sans-extension' are defined in both (guix utils) and
>>> (guix build guile-build-system). They are both the same. Should we move
>>> them to (guix build utils) as well?
>>
>> Yes, sounds like a good idea!
>
> ‘file-sans-extension’ is used on the host-side, e.g. in scripts/build.scm.
> Is it still fine to move it to a build-side module only?

I think so because in practice we already use (guix build utils) quite a
lot on the “host side”.

More generally, while I think it’s important to arrange things so that
we don’t end up importing all of Guix on the build side, the advantage
of using the same language on both sides is that we can share code.  I
think we should take advantage of it when there are no downsides.

Ludo’.