unofficial mirror of guix-devel@gnu.org 
 help / color / mirror / code / Atom feed
* Re: 02/02: gnu: libical: Add upstream patch to preserve API compatibility.
       [not found] ` <20200203142559.6BA8E20A10@vcs0.savannah.gnu.org>
@ 2020-02-04 22:31   ` Ludovic Courtès
  2020-02-05 21:29     ` Marius Bakke
  0 siblings, 1 reply; 4+ messages in thread
From: Ludovic Courtès @ 2020-02-04 22:31 UTC (permalink / raw)
  To: Marius Bakke; +Cc: Guix-devel

Howdy!

guix-commits@gnu.org skribis:

> +               (list (origin
> +                       (method url-fetch)
> +                       (uri (string-append
> +                             "https://github.com/libical/libical/commit/"
> +                             "ae394010c889e4c185160da5e81527849f9de350.patch"))

IIRC, those generated patches are not stable, like generated tarballs,
no?  What are the available options?

Thanks,
Ludo’.

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

* Re: 02/02: gnu: libical: Add upstream patch to preserve API compatibility.
  2020-02-04 22:31   ` 02/02: gnu: libical: Add upstream patch to preserve API compatibility Ludovic Courtès
@ 2020-02-05 21:29     ` Marius Bakke
  2020-02-07 21:44       ` Ludovic Courtès
  0 siblings, 1 reply; 4+ messages in thread
From: Marius Bakke @ 2020-02-05 21:29 UTC (permalink / raw)
  To: Ludovic Courtès; +Cc: Guix-devel

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

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

> Howdy!
>
> guix-commits@gnu.org skribis:
>
>> +               (list (origin
>> +                       (method url-fetch)
>> +                       (uri (string-append
>> +                             "https://github.com/libical/libical/commit/"
>> +                             "ae394010c889e4c185160da5e81527849f9de350.patch"))
>
> IIRC, those generated patches are not stable, like generated tarballs,
> no?  What are the available options?

I haven't seen GitHub change such generated patch files yet[*], but I do
agree it's "fragile", because they can be difficult to reproduce when
GitHub inevitably disappears, or _do_ change the format.

I wonder if it's feasible to use Guile-Git to generate the patches
instead, at the expense of having to download the entire repository
for the purpose of extracting one or more (substitutable) patches.

[*] There was one instance where it had changed, but that was a merge
commit(!), which is a pretty extreme case (whoops):
https://git.savannah.gnu.org/cgit/guix.git/commit/?id=7670efefe4fb4aca12cb19ea5d89ff37c48e3ea6

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

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

* Re: 02/02: gnu: libical: Add upstream patch to preserve API compatibility.
  2020-02-05 21:29     ` Marius Bakke
@ 2020-02-07 21:44       ` Ludovic Courtès
  2020-02-10  8:16         ` zimoun
  0 siblings, 1 reply; 4+ messages in thread
From: Ludovic Courtès @ 2020-02-07 21:44 UTC (permalink / raw)
  To: Marius Bakke; +Cc: Guix-devel

Hi!

Marius Bakke <mbakke@fastmail.com> skribis:

> Ludovic Courtès <ludo@gnu.org> writes:
>
>> Howdy!
>>
>> guix-commits@gnu.org skribis:
>>
>>> +               (list (origin
>>> +                       (method url-fetch)
>>> +                       (uri (string-append
>>> +                             "https://github.com/libical/libical/commit/"
>>> +                             "ae394010c889e4c185160da5e81527849f9de350.patch"))
>>
>> IIRC, those generated patches are not stable, like generated tarballs,
>> no?  What are the available options?
>
> I haven't seen GitHub change such generated patch files yet[*], but I do
> agree it's "fragile", because they can be difficult to reproduce when
> GitHub inevitably disappears, or _do_ change the format.

Yeah, I remember Cgit would include its version number in patches, for
instance.

> I wonder if it's feasible to use Guile-Git to generate the patches
> instead, at the expense of having to download the entire repository
> for the purpose of extracting one or more (substitutable) patches.

Good question.  We could have a lowerable <git-commit> data type that
would (1) clone the repo, and (2) dump the raw (?) commit to #$output.
It’s happening on the build side, like ‘git-fetch’, so it could use Git
directly, which supports shallow clones.

Sounds like it could work!

Ludo’.

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

* Re: 02/02: gnu: libical: Add upstream patch to preserve API compatibility.
  2020-02-07 21:44       ` Ludovic Courtès
@ 2020-02-10  8:16         ` zimoun
  0 siblings, 0 replies; 4+ messages in thread
From: zimoun @ 2020-02-10  8:16 UTC (permalink / raw)
  To: Ludovic Courtès; +Cc: Guix-devel

Hi,

On Fri, 7 Feb 2020 at 22:44, Ludovic Courtès <ludo@gnu.org> wrote:

> Marius Bakke <mbakke@fastmail.com> skribis:


> > I wonder if it's feasible to use Guile-Git to generate the patches
> > instead, at the expense of having to download the entire repository
> > for the purpose of extracting one or more (substitutable) patches.
>
> Good question.  We could have a lowerable <git-commit> data type that
> would (1) clone the repo, and (2) dump the raw (?) commit to #$output.
> It’s happening on the build side, like ‘git-fetch’, so it could use Git
> directly, which supports shallow clones.

I do not know if it makes sense but Git provides now a (experimental)
partial download/clone.

https://git-scm.com/docs/partial-clone


All the best,
simon

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

end of thread, other threads:[~2020-02-10  8:17 UTC | newest]

Thread overview: 4+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
     [not found] <20200203142557.25275.13298@vcs0.savannah.gnu.org>
     [not found] ` <20200203142559.6BA8E20A10@vcs0.savannah.gnu.org>
2020-02-04 22:31   ` 02/02: gnu: libical: Add upstream patch to preserve API compatibility Ludovic Courtès
2020-02-05 21:29     ` Marius Bakke
2020-02-07 21:44       ` Ludovic Courtès
2020-02-10  8:16         ` zimoun

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