-
Notifications
You must be signed in to change notification settings - Fork 13.6k
[llvm] remove unusable llvm.spec.in #96825
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Conversation
The `llvm.spec.in` is turned into `llvm.spec` through cmake. The spec file's `%build` section runs `./configure` which has been deprecated since 2016 (See e49730d).
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
The code is broken and fails to generate the SRPM.
Even if generated a correct SRPM, it would still fail because the spec file still uses llvm/configure
.
This is a clear indication nobody has been using it.
LGTM.
LLVM Buildbot has detected a new failure on builder Full details are available at: https://lab.llvm.org/buildbot/#/builders/51/builds/744 Here is the relevant piece of the build log for the reference:
|
I think the failure is unrelated to this change. @tuliom any thoughts? |
@kwk Agreed. It looks like this test is failing intermittently, e.g. https://lab.llvm.org/buildbot/#/builders/51/builds/691 |
The `llvm.spec.in` is turned into `llvm.spec` through cmake. The spec file's `%build` section runs `./configure` which has been deprecated since 2016 (See e49730d).
The `llvm.spec.in` is turned into `llvm.spec` through cmake. The spec file's `%build` section runs `./configure` which has been deprecated since 2016 (See e49730d).
The
llvm.spec.in
is turned intollvm.spec
through cmake. The spec file's%build
section runs./configure
which has been deprecated since 2016 (See e49730d).