unofficial mirror of bug-guix@gnu.org 
 help / color / mirror / code / Atom feed
* bug#47230: Build phase to graft during build for better grafts QA
@ 2021-03-18 11:38 Léo Le Bouter via Bug reports for GNU Guix
  2021-03-18 20:41 ` Ludovic Courtès
  0 siblings, 1 reply; 3+ messages in thread
From: Léo Le Bouter via Bug reports for GNU Guix @ 2021-03-18 11:38 UTC (permalink / raw)
  To: 47230

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

Hello!

I am having an hard time testing grafts in GNU Guix while I think we
could have better tooling around this.

For example, we could have a package transformation that can add a
phase before 'check (or others) to graft any intermediate build binary
and all dependencies (if not done already) to run the test suite WITH
the graft. This could be turned on by GNU Guix contributors after
trying to graft some package to ensure it does not break things.

What do you think?

Léo

[-- Attachment #2: This is a digitally signed message part --]
[-- Type: application/pgp-signature, Size: 833 bytes --]

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

* bug#47230: Build phase to graft during build for better grafts QA
  2021-03-18 11:38 bug#47230: Build phase to graft during build for better grafts QA Léo Le Bouter via Bug reports for GNU Guix
@ 2021-03-18 20:41 ` Ludovic Courtès
  2021-03-19  9:44   ` Léo Le Bouter via Bug reports for GNU Guix
  0 siblings, 1 reply; 3+ messages in thread
From: Ludovic Courtès @ 2021-03-18 20:41 UTC (permalink / raw)
  To: Léo Le Bouter; +Cc: 47230

Hi,

Léo Le Bouter <lle-bout@zaclys.net> skribis:

> I am having an hard time testing grafts in GNU Guix while I think we
> could have better tooling around this.
>
> For example, we could have a package transformation that can add a
> phase before 'check (or others) to graft any intermediate build binary
> and all dependencies (if not done already) to run the test suite WITH
> the graft. This could be turned on by GNU Guix contributors after
> trying to graft some package to ensure it does not break things.
>
> What do you think?

I think it’s more of a discussion for guix-devel than a bug report.  :-)

What you describe, AIUI, is not possible: there are no phases or
anything like that happening on grafted packages.  Quoth the manual
(info "(guix) Security Updates"):

  Other restrictions may apply: for instance, when adding a graft to a
  package providing a shared library, the original shared library and its
  replacement must have the same ‘SONAME’ and be binary-compatible.

As I wrote earlier today, these things have to be checked by packagers;
they’re not easily automated because that usually involves knowing the
intent of upstream developers, for example whether they intend the new
version to be ABI-compatible with the version we have at hand, etc.

HTH!

Ludo’.




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

* bug#47230: Build phase to graft during build for better grafts QA
  2021-03-18 20:41 ` Ludovic Courtès
@ 2021-03-19  9:44   ` Léo Le Bouter via Bug reports for GNU Guix
  0 siblings, 0 replies; 3+ messages in thread
From: Léo Le Bouter via Bug reports for GNU Guix @ 2021-03-19  9:44 UTC (permalink / raw)
  To: Ludovic Courtès; +Cc: 47230

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

On Thu, 2021-03-18 at 21:41 +0100, Ludovic Courtès wrote:
> I think it’s more of a discussion for guix-devel than a bug
> report.  :-)

Yes but then I was thinking how do we track progress without losing it
in the pile of emails from guix-devel people receive everyday which
made me create a bug in the idea of "feature-request".. don't know.

> What you describe, AIUI, is not possible: there are no phases or
> anything like that happening on grafted packages.  Quoth the manual
> (info "(guix) Security Updates"):
> 
>   Other restrictions may apply: for instance, when adding a graft to
> a
>   package providing a shared library, the original shared library and
> its
>   replacement must have the same ‘SONAME’ and be binary-compatible.

I am not sure we understand each other, I am proposing we could add
tooling to aid that testing.

> As I wrote earlier today, these things have to be checked by
> packagers;
> they’re not easily automated because that usually involves knowing
> the
> intent of upstream developers, for example whether they intend the
> new
> version to be ABI-compatible with the version we have at hand, etc.
> 
> HTH!

Yes of course, but aided by tooling that process can me smoother, e.g.
figuring out the SONAME has changed could at least be checked
automatically and issue a warning/error. Same thing with abidiff etc.
these tools are really great and we could create an easy way to run the
test suite of a package with the graft in also.

> Ludo’.

Thank you

[-- Attachment #2: This is a digitally signed message part --]
[-- Type: application/pgp-signature, Size: 833 bytes --]

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

end of thread, other threads:[~2021-03-19  9:46 UTC | newest]

Thread overview: 3+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2021-03-18 11:38 bug#47230: Build phase to graft during build for better grafts QA Léo Le Bouter via Bug reports for GNU Guix
2021-03-18 20:41 ` Ludovic Courtès
2021-03-19  9:44   ` Léo Le Bouter via Bug reports for GNU Guix

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