all messages for Guix-related lists mirrored at yhetil.org
 help / color / mirror / code / Atom feed
* [bug#57858] [PATCH] gnu: git-annex: Promote git to propagated input
@ 2022-09-16 12:37 Konrad Hinsen
  2022-09-16 15:02 ` Maxime Devos
  0 siblings, 1 reply; 8+ messages in thread
From: Konrad Hinsen @ 2022-09-16 12:37 UTC (permalink / raw)
  To: 57858

* gnu/packages/haskell-apps.scm (git-annex): changed inputs
[propagated-inputs]: Add git
[inputs]: Remove git
---
 gnu/packages/haskell-apps.scm | 3 ++-
 1 file changed, 2 insertions(+), 1 deletion(-)

diff --git a/gnu/packages/haskell-apps.scm b/gnu/packages/haskell-apps.scm
index 50380281d7..d5e855cf9a 100644
--- a/gnu/packages/haskell-apps.scm
+++ b/gnu/packages/haskell-apps.scm
@@ -462,8 +462,9 @@ (define-public git-annex
            ghc-utf8-string
            ghc-uuid
            ghc-vector
-           git
            rsync))
+    (propagated-inputs
+     (list git))
     (native-inputs
      (list ghc-tasty ghc-tasty-hunit ghc-tasty-quickcheck ghc-tasty-rerun
            perl))
-- 
2.37.3




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

* [bug#57858] [PATCH] gnu: git-annex: Promote git to propagated input
  2022-09-16 12:37 [bug#57858] [PATCH] gnu: git-annex: Promote git to propagated input Konrad Hinsen
@ 2022-09-16 15:02 ` Maxime Devos
  2022-09-17  9:56   ` Konrad Hinsen
  0 siblings, 1 reply; 8+ messages in thread
From: Maxime Devos @ 2022-09-16 15:02 UTC (permalink / raw)
  To: Konrad Hinsen, 57858


[-- Attachment #1.1.1: Type: text/plain, Size: 669 bytes --]

On 16-09-2022 14:37, Konrad Hinsen wrote:
> * gnu/packages/haskell-apps.scm (git-annex): changed inputs
> [propagated-inputs]: Add git
> [inputs]: Remove git
> ---
>   gnu/packages/haskell-apps.scm | 3 ++-
>   1 file changed, 2 insertions(+), 1 deletion(-)

What's the reason for this patch?  My guess is that it's because 
git-annex cannot find git:

$ guix shell --pure git-annex -- git-annex test
git-annex: git: createProcess: posix_spawnp: does not exist (No such 
file or directory)

If so, propagation (which as a few small downsides) can be avoided with 
an appropriate substitute* fixing the file name of the git binary.

Greetings,
Maxime

[-- Attachment #1.1.2: OpenPGP public key --]
[-- Type: application/pgp-keys, Size: 929 bytes --]

[-- Attachment #2: OpenPGP digital signature --]
[-- Type: application/pgp-signature, Size: 236 bytes --]

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

* [bug#57858] [PATCH] gnu: git-annex: Promote git to propagated input
  2022-09-16 15:02 ` Maxime Devos
@ 2022-09-17  9:56   ` Konrad Hinsen
  2022-10-17  8:07     ` bug#57858: " Ludovic Courtès
  0 siblings, 1 reply; 8+ messages in thread
From: Konrad Hinsen @ 2022-09-17  9:56 UTC (permalink / raw)
  To: Maxime Devos, 57858

Maxime Devos <maximedevos@telenet.be> writes:

> What's the reason for this patch?  My guess is that it's because 
> git-annex cannot find git:

That was indeed the starting point. But...

> If so, propagation (which as a few small downsides) can be avoided with 
> an appropriate substitute* fixing the file name of the git binary.

... I tried that and still got error messages whose exact cause I don't
quite understand.

What made me opt for git as a propagated input is that (1) it works just
fine and (2) there isn't much you can do with git-annex without having
git as well. git is even the official user interface of git-annex.

Cheers,
  Konrad




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

* bug#57858: [PATCH] gnu: git-annex: Promote git to propagated input
  2022-09-17  9:56   ` Konrad Hinsen
@ 2022-10-17  8:07     ` Ludovic Courtès
  2022-10-17  9:29       ` [bug#57858] " zimoun
  0 siblings, 1 reply; 8+ messages in thread
From: Ludovic Courtès @ 2022-10-17  8:07 UTC (permalink / raw)
  To: Konrad Hinsen; +Cc: 57858-done, Maxime Devos

Hi,

Konrad Hinsen <konrad.hinsen@fastmail.net> skribis:

> Maxime Devos <maximedevos@telenet.be> writes:
>
>> What's the reason for this patch?  My guess is that it's because 
>> git-annex cannot find git:
>
> That was indeed the starting point. But...
>
>> If so, propagation (which as a few small downsides) can be avoided with 
>> an appropriate substitute* fixing the file name of the git binary.
>
> ... I tried that and still got error messages whose exact cause I don't
> quite understand.
>
> What made me opt for git as a propagated input is that (1) it works just
> fine and (2) there isn't much you can do with git-annex without having
> git as well. git is even the official user interface of git-annex.

Yeah, that sounds reasonable: I’d have made the same suggestion as
Maxime but any improvement here is welcome.

Applied, thanks!

Ludo’.




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

* [bug#57858] [PATCH] gnu: git-annex: Promote git to propagated input
  2022-10-17  8:07     ` bug#57858: " Ludovic Courtès
@ 2022-10-17  9:29       ` zimoun
  2022-10-17 10:11         ` Konrad Hinsen
  0 siblings, 1 reply; 8+ messages in thread
From: zimoun @ 2022-10-17  9:29 UTC (permalink / raw)
  To: Ludovic Courtès, Konrad Hinsen; +Cc: 57858-done, Maxime Devos

Hi,

On lun., 17 oct. 2022 at 10:07, Ludovic Courtès <ludo@gnu.org> wrote:

>> What made me opt for git as a propagated input is that (1) it works just
>> fine and (2) there isn't much you can do with git-annex without having
>> git as well. git is even the official user interface of git-annex.
>
> Yeah, that sounds reasonable: I’d have made the same suggestion as
> Maxime but any improvement here is welcome.

We are usually not propagating.  For example, emacs-magit or git-lfs or
etc.

From my understanding, this change is not coherent with other parts.

About (2), what do you mean Konrad?  Is ’git-annex’ not working as
expected?  Well, ’git annex’ is a shorthand for ’git-annex’ and I
thought this ’git-annex’ is self-contained; for which operations
’git-annex’ depends on the ’git’ binaires?

Cheers,
simon




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

* [bug#57858] [PATCH] gnu: git-annex: Promote git to propagated input
  2022-10-17  9:29       ` [bug#57858] " zimoun
@ 2022-10-17 10:11         ` Konrad Hinsen
  2022-10-17 11:02           ` zimoun
  2022-10-17 12:55           ` Maxime Devos
  0 siblings, 2 replies; 8+ messages in thread
From: Konrad Hinsen @ 2022-10-17 10:11 UTC (permalink / raw)
  To: zimoun, Ludovic Courtès; +Cc: 57858-done, Maxime Devos

zimoun <zimon.toutoune@gmail.com> writes:

> About (2), what do you mean Konrad?  Is ’git-annex’ not working as
> expected?  Well, ’git annex’ is a shorthand for ’git-annex’ and I
> thought this ’git-annex’ is self-contained; for which operations
> ’git-annex’ depends on the ’git’ binaires?

Maxime quoted a simple command line example. git-annex requires git for
pretty much everything it does.

But that's not the question here. As Maxime said, git could just be a
standard (non-propagated) input with a small patch to git-annex. That
makes it work to a large degree (requiring you to replace "git annex" by
"git-annex" in all command lines), but it occasionally failed for me and
I didn't invest much effort to explore why, given that I consider it
pointless to try to use git-annex without git.

But you are right in pointing out that this is an instance of a
situation for which Guix should have a general policy. git-lfs is
probably in exactly the same situation. Not emacs-magit though: it works
perfectly fine without exposing git in the same profile. Being a git
plugin is not the same as being a git wrapper.

Cheers,
  Konrad




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

* [bug#57858] [PATCH] gnu: git-annex: Promote git to propagated input
  2022-10-17 10:11         ` Konrad Hinsen
@ 2022-10-17 11:02           ` zimoun
  2022-10-17 12:55           ` Maxime Devos
  1 sibling, 0 replies; 8+ messages in thread
From: zimoun @ 2022-10-17 11:02 UTC (permalink / raw)
  To: Konrad Hinsen, Ludovic Courtès; +Cc: 57858-done, Maxime Devos

Hi Konrad,

On lun., 17 oct. 2022 at 12:11, Konrad Hinsen <konrad.hinsen@fastmail.net> wrote:

> But you are right in pointing out that this is an instance of a
> situation for which Guix should have a general policy. git-lfs is
> probably in exactly the same situation. Not emacs-magit though: it works
> perfectly fine without exposing git in the same profile. Being a git
> plugin is not the same as being a git wrapper.

From my understanding, the general policy is to not propagate git. :-)

From gnu/packages/version-control.scm, where xyz: is the line number

--8<---------------cut here---------------start------------->8---
778:(define-public git-cal                     no inputs
907:(define-public git-crypt                   inputs: git
960:(define-public git-remote-gcrypt           no inputs
1108:(define-public python-git-multimail       inputs: git
1311:(define-public git-flow                   inputs: no git
1451:(define-public git-test-sequence          no inputs
2681:(define-public git-annex-remote-rclone    inputs: no git
2901:(define-public git-when-merged            inputs: git
2952:(define-public git-imerge                 inputs: git
2997:(define-public git-lfs                    native-inputs only and no git
3046:(define-public git-open                   inputs: no git
3392:(define-public git-filter-repo            inputs: no git
--8<---------------cut here---------------end--------------->8---

The package git-repo from (gnu packages android) contains git as
inputs.  The package git-absord from (gnu packages rust-apps) does not
contains git as inputs.


I understand the issue but I think the propagation is not the correct
solution. :-)


Cheers,
simon





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

* [bug#57858] [PATCH] gnu: git-annex: Promote git to propagated input
  2022-10-17 10:11         ` Konrad Hinsen
  2022-10-17 11:02           ` zimoun
@ 2022-10-17 12:55           ` Maxime Devos
  1 sibling, 0 replies; 8+ messages in thread
From: Maxime Devos @ 2022-10-17 12:55 UTC (permalink / raw)
  To: Konrad Hinsen, zimoun, Ludovic Courtès; +Cc: 57858-done


[-- Attachment #1.1.1: Type: text/plain, Size: 833 bytes --]



On 17-10-2022 12:11, Konrad Hinsen wrote:
> But you are right in pointing out that this is an instance of a
> situation for which Guix should have a general policy. git-lfs is
> probably in exactly the same situation. Not emacs-magit though: it works
> perfectly fine without exposing git in the same profile. Being a git
> plugin is not the same as being a git wrapper.

The current policy, IMO, is that plugins don't propagate what they are 
plugged into -- even if the plugin is useless without the thing it 
plugs-in into (compare: search paths are set in 'consumers', not 
'producers').

Taking 'emacs-magit' as an example, emacs-magit is an Emacs plugin, but 
it doesn't propagate emacs.

Likewise, (in some usages) 'git-annex' is a git plugin, yet it (IMO) 
shouldn't propagate git.

Greetings
Maxime.

[-- Attachment #1.1.2: OpenPGP public key --]
[-- Type: application/pgp-keys, Size: 929 bytes --]

[-- Attachment #2: OpenPGP digital signature --]
[-- Type: application/pgp-signature, Size: 236 bytes --]

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

end of thread, other threads:[~2022-10-17 12:56 UTC | newest]

Thread overview: 8+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2022-09-16 12:37 [bug#57858] [PATCH] gnu: git-annex: Promote git to propagated input Konrad Hinsen
2022-09-16 15:02 ` Maxime Devos
2022-09-17  9:56   ` Konrad Hinsen
2022-10-17  8:07     ` bug#57858: " Ludovic Courtès
2022-10-17  9:29       ` [bug#57858] " zimoun
2022-10-17 10:11         ` Konrad Hinsen
2022-10-17 11:02           ` zimoun
2022-10-17 12:55           ` Maxime Devos

Code repositories for project(s) associated with this external index

	https://git.savannah.gnu.org/cgit/guix.git

This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.