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
>
>
>
>
?