Skip to content
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

Open
wants to merge 1 commit into
base: master
Choose a base branch
from
Open
Show file tree
Hide file tree
Changes from all commits
Commits
File filter

Filter by extension

Filter by extension

Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
1 change: 1 addition & 0 deletions net-fs/eos-client/Manifest
Original file line number Diff line number Diff line change
@@ -0,0 +1 @@
DIST 5.1.27.tar.gz 101888711 BLAKE2B f5dfd4ad430005351aa1a4c21932b22ff2a95a4970d5119933385785c3c29286b9a994f38c8d3d9de1ab653692fd8492ce850d692ebc6dcbde1534300f4004c1 SHA512 c4e2aace5bdb57cfa176560028ddcd08be3c15eae6880b99901895f30000029a7fff14d8b2e50b01fc4d3d064e74d44c0d5b15226174bd771cd6cc92f5e8a5d3
79 changes: 79 additions & 0 deletions net-fs/eos-client/eos-client-5.1.27.ebuild
Original file line number Diff line number Diff line change
@@ -0,0 +1,79 @@
# Copyright 1999-2023 Gentoo Authors
# Distributed under the terms of the GNU General Public License v2

EAPI=8

inherit cmake

DESCRIPTION="EOS Storage"
HOMEPAGE="https://eos-web.web.cern.ch/eos-web/"
SRC_URI="https://github.com/cern-eos/eos/archive/refs/tags/${PV}.tar.gz"
LICENSE="GPL-3"

SLOT="0"
KEYWORDS="~amd64"
Copy link
Member

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)


S=${WORKDIR}/eos-${PV}

IUSE="jemalloc rocksdb httpd"

RDEPEND="
net-misc/curl
sys-libs/zlib
dev-libs/jsoncpp
dev-libs/openssl
net-libs/xrootd[http,kerberos]
sys-libs/binutils-libs
sys-libs/ncurses
sys-apps/util-linux
sys-fs/fuse
sys-libs/readline
dev-libs/leveldb
net-libs/zeromq
virtual/krb5
dev-libs/libevent
app-arch/bzip2
dev-libs/xxhash
net-libs/davix
dev-libs/protobuf
jemalloc? ( dev-libs/jemalloc )
rocksdb? (
dev-libs/rocksdb
app-arch/zstd
app-arch/lz4
)
httpd? ( net-libs/libmicrohttpd )
"

DEPEND="${RDEPEND}"

BDEPEND="
dev-lang/python
Copy link
Member

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.

dev-cpp/sparsehash
dev-util/pkgconf
net-libs/cppzmq
sys-apps/help2man
sys-fs/xfsprogs
dev-cpp/gtest
"

src_prepare() {
default

cmake_src_prepare
Copy link
Member

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_configure() {
local mycmakeargs=(
-DCLIENT=on
)
cmake_src_configure
}

src_compile() {
cmake_src_compile
}

src_install() {
cmake_src_install
}
Copy link
Member

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.

Loading