Stray po.go file

  • Open
  • quality assurance status badge
Details
2 participants
  • Andreas Enge
  • Julien Lepiller
Owner
unassigned
Submitted by
Andreas Enge
Severity
normal
A
A
Andreas Enge wrote on 21 Apr 2023 15:00
(address . bug-guix@gnu.org)
ZEKJBrMo01Z/pDFN@jurong
This is not related to the dancefloor. Someone on IRC mentioned that
"make clean-go" leaves a stray file guix/build/po.go.

I suppose that it should be added to Makefile.am in the line
that currently reads:
GOBJECTS = $(MODULES:%.scm=%.go) guix/config.go $(dist_noinst_DATA:%.scm=%.go)

but would like to leave it to the experts.

Actually further above we have this:
MODULES_NOT_COMPILED = \
guix/build/po.scm \
guix/man-db.scm

So maybe a better fix would be to move guix/build/po.scm to MODULES
instead, since apparently it is compiled.

Andreas
J
J
Julien Lepiller wrote on 21 Apr 2023 18:04
A5367E89-27FE-4184-9D82-F845677D8836@lepiller.eu
I don't remember why we did that. The constraints for that file are:

- it's not required to install it
- it must be built before the manuals are generated (it's used for translations)
- it should be built, since otherwise it takes a long time to run
- it has no dependencies
- it's not actually stray

Not sure what the best solution for that is.

Le 21 avril 2023 15:00:54 GMT+02:00, Andreas Enge <andreas@enge.fr> a écrit :
Toggle quote (22 lines)
>This is not related to the dancefloor. Someone on IRC mentioned that
>"make clean-go" leaves a stray file guix/build/po.go.
>
>I suppose that it should be added to Makefile.am in the line
>that currently reads:
>GOBJECTS = $(MODULES:%.scm=%.go) guix/config.go $(dist_noinst_DATA:%.scm=%.go)
>
>but would like to leave it to the experts.
>
>Actually further above we have this:
>MODULES_NOT_COMPILED = \
> guix/build/po.scm \
> guix/man-db.scm
>
>So maybe a better fix would be to move guix/build/po.scm to MODULES
>instead, since apparently it is compiled.
>
>Andreas
>
>
>
>
?
Your comment

Commenting via the web interface is currently disabled.

To comment on this conversation send an email to 62992@debbugs.gnu.org

To respond to this issue using the mumi CLI, first switch to it
mumi current 62992
Then, you may apply the latest patchset in this issue (with sign off)
mumi am -- -s
Or, compose a reply to this issue
mumi compose
Or, send patches to this issue
mumi send-email *.patch