Skip to content
This repository was archived by the owner on Jul 6, 2023. It is now read-only.

Conversation

@fickludd
Copy link
Contributor

@fickludd fickludd commented Apr 4, 2019

No description provided.

out/rpm8/RPMS/noarch/$(rpm8file): out/rpm8/SPECS/cypher-shell.spec $(rpm_artifacts) out/rpm8/BUILD/Makefile out/rpm8/BUILD/cypher-shell.1.md
rpmbuild --define "_topdir $(CURDIR)/out/rpm8" -bb --clean $<

.PHONY: rpm
Copy link
Member

Choose a reason for hiding this comment

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

There's now no longer a make target for rpm, so you can't do make rpm any more.

add a line somewhere:
rpm: rpm8 rpm11

Then just do

```
make debian rpm
Copy link
Member

Choose a reason for hiding this comment

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

again, let's keep the rpm target

To test the packages you need to have Docker installed:

```
make debian-test rpm-test
Copy link
Member

Choose a reason for hiding this comment

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

and rpm-test


Requires: which, (jre-headless >= 1.8 or jre >= 11)
Conflicts: cypher-shell-java8
Requires: which, jre >= 11
Copy link
Member

Choose a reason for hiding this comment

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

Add an Obsoletes here too?

@fickludd fickludd changed the title Branch into 2 rpm packages Branch into 3 rpm packages Apr 8, 2019
@fickludd fickludd merged commit ba67fab into neo4j:master Apr 8, 2019
@fickludd fickludd deleted the new-rpm-packaging branch April 8, 2019 09:13
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants