-
Notifications
You must be signed in to change notification settings - Fork 165
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
net-fs/eos-client: new package #1212
base: master
Are you sure you want to change the base?
Conversation
The code of |
The |
There are plenty of submodules of git that needs to be downloaded and extracted before heading to |
Most of them could be replaced by the system dependencies without changes. The cmake script of eos seems clever enough. |
src_prepare() { | ||
default | ||
|
||
cmake_src_prepare |
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.
Both default
and cmake_src_prepare
are going to try and apply any user patches. The second try will cause the build to fails since the patches are already applied. You can probably drop the entire explicit src_prepare
phase.
|
||
src_install() { | ||
cmake_src_install | ||
} |
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.
You don't have to set src_compile
and src_install
if you're only using the standard function.
LICENSE="GPL-3" | ||
|
||
SLOT="0" | ||
KEYWORDS="~amd64" |
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.
Try to keep the layout as close to the ebuild skeleton as possible. (i.e. move LICENSE
one line down)
DEPEND="${RDEPEND}" | ||
|
||
BDEPEND=" | ||
dev-lang/python |
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.
If this requires python during build time, it should be inheriting python-any-r1.eclass
and setting PYTHON_COMPAT
to the supported python versions.
I think for
Lets use system versions whenever possible. |
/etc
directory.