Skip to content

Emit patches for a given Tow-Boot build#22

Merged
samueldr merged 9 commits intoTow-Boot:developmentfrom
samueldr:feature/emit-patches
May 23, 2021
Merged

Emit patches for a given Tow-Boot build#22
samueldr merged 9 commits intoTow-Boot:developmentfrom
samueldr:feature/emit-patches

Conversation

@samueldr
Copy link
Contributor

The outputs, which in turn are used for the release tarballs, now includes a patches directory with all the patches applied on top of U-Boot.

Combine this with the fact the .config file is available in the root of the tarball, it should be easier for individuals to re-combine a given Tow-Boot release on top of U-Boot without using the tooling here.

Note that this is not meant to replace the tooling. This is mainly to better handle GPL compliance by providing all patches side-by-side with the binaries.

samueldr added 9 commits May 23, 2021 15:32
This provides a `series` folder with all the patches needed for
reproducing the build.
These commands are the common parts for the different builds. This way
it will be easier in the future to add things to the build.
Comment on lines +284 to +286
${lib.concatMapStringsSep "\n" (p: "cp ${p} ./${baseNameOf (toString p)}") tow-boot.patches}
cat <<EOF > series
${lib.concatMapStringsSep "\n" (p: "${baseNameOf (toString p)}") tow-boot.patches}
Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This will break if two patches end up named the same.

But it's not much of an issue as if two patches end up having the same exact name, it will break loudly since the cp will fail overwriting the file.

@samueldr samueldr merged commit 01241cf into Tow-Boot:development May 23, 2021
@samueldr samueldr deleted the feature/emit-patches branch May 23, 2021 20:38
@samueldr samueldr added this to the "next" milestone Jun 21, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Development

Successfully merging this pull request may close these issues.

1 participant