mbox series

[bug#69337,0/2] guile-build-system: Install .scm before .go.

Message ID cover.1708709549.git.~@wolfsden.cz
Headers show
Series guile-build-system: Install .scm before .go. | expand

Message

Tomas Volf Feb. 23, 2024, 5:32 p.m. UTC 
Currently the .go files are generated before installing the accompanying .scm
files, which leads to noise in the build log (source file ... newer than).
Swapping the order of the actions fixes that.

The indentation was also wrong, so I took liberty to fix that first in a
separate commit, so that the second diff is clean.

Tomas Volf (2):
  build-system/guile: Fix indentation.
  build-system/guile: Install .scm files first.

 guix/build/guile-build-system.scm | 41 +++++++++++++++----------------
 1 file changed, 20 insertions(+), 21 deletions(-)


base-commit: aefc7428203203ae88c64cc4769113453c01a185
--
2.41.0

Comments

Ludovic Courtès March 2, 2024, 3:28 p.m. UTC | #1 
Hi,

Tomas Volf <~@wolfsden.cz> skribis:

> Currently the .go files are generated before installing the accompanying .scm
> files, which leads to noise in the build log (source file ... newer than).
> Swapping the order of the actions fixes that.
>
> The indentation was also wrong, so I took liberty to fix that first in a
> separate commit, so that the second diff is clean.
>
> Tomas Volf (2):
>   build-system/guile: Fix indentation.
>   build-system/guile: Install .scm files first.

Applied, thanks!

Ludo’.