[PATCH] D101446: [llvm] Improve export.sh with help and snapshot

Sun May 9 13:27:08 PDT 2021

kwk added a comment.

In D101446#2745415 <https://reviews.llvm.org/D101446#2745415>, @aaronpuchert wrote:

> Somehow the patch got messed up, I think. The changes that I see look like the changes since the previous patch, not the new patch.

This should be fixed now. Sorry for the mess.

> In D101446#2743089 <https://reviews.llvm.org/D101446#2743089>, @kwk wrote:
>
>> Like I mentioned to @hans above, I think we could utilize github to generate daily source tarballs for us and stored them in a pre-release.
>
> So you want to build daily snapshots as RPM, and for that need the sources to be on GitHub to have them official?

In short, yes.

>> Sorry to bore you with these RPM stuff. But I think it proofs a lot of benefit.
>
> Not boring me, openSUSE works with RPM as well. Though our build service can generate tarballs from git repositories <https://github.com/openSUSE/obs-service-tar_scm> automatically, so if I wanted to build snapshots I would probably just use that.

Generating the tarballs takes twelve minutes or so. For myself I wanted to have this outsourced of the build pipeline for the RPMs. From my short past experience with building snapshots, it wasn't LLVM in particular that needed to be adjusted in order for a build to go through. It was more about fixing paths or removing patches that already landed upstream. The feedback time for building an RPM that eventually fails was drastically reduced for me by relying on pre-built source tarballs. One other great benefit is that I can re-use the same RPM spec file for building snapshots and regular releases. This makes it easy to track changes that go into the next release more easy.

In D101446#2745416 <https://reviews.llvm.org/D101446#2745416>, @aaronpuchert wrote:

> (I think you have to squash the changes, Arcanist seems to take only the last commit.)

Please let me know if this is fixed now.

Repository:
  rG LLVM Github Monorepo

CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D101446/new/

https://reviews.llvm.org/D101446