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

sys-block/targetcli-fb: add dbus dependency / bump to EAPI 7 #8972

Closed
wants to merge 10,000 commits into from

Conversation

diogocp
Copy link
Contributor

@diogocp diogocp commented Jun 25, 2018

perfinion and others added 30 commits June 25, 2018 18:40
Gentoo-bug: 658956
Package-Manager: Portage-2.3.40, Repoman-2.3.9
Gentoo-bug: 658956
Package-Manager: Portage-2.3.40, Repoman-2.3.9
Gentoo-bug: 658956
Package-Manager: Portage-2.3.40, Repoman-2.3.9
Gentoo-bug: 658956
Package-Manager: Portage-2.3.40, Repoman-2.3.9
Gentoo-bug: 658956
Package-Manager: Portage-2.3.40, Repoman-2.3.9
Gentoo-bug: 658956
Package-Manager: Portage-2.3.40, Repoman-2.3.9
Gentoo-bug: 658956
Package-Manager: Portage-2.3.40, Repoman-2.3.9
Gentoo-bug: 658956
Package-Manager: Portage-2.3.40, Repoman-2.3.9
Gentoo-bug: 658956
Package-Manager: Portage-2.3.40, Repoman-2.3.9
Package-Manager: Portage-2.3.40, Repoman-2.3.9
Package-Manager: Portage-2.3.40, Repoman-2.3.9
Package-Manager: Portage-2.3.40, Repoman-2.3.9
RepoMan-Options: --ignore-arches
Package-Manager: Portage-2.3.40, Repoman-2.3.9
Package-Manager: Portage-2.3.40, Repoman-2.3.9
Closes: https://bugs.gentoo.org/650050
Package-Manager: Portage-2.3.40, Repoman-2.3.9
Package-Manager: Portage-2.3.40, Repoman-2.3.9
Package-Manager: Portage-2.3.40, Repoman-2.3.9
Whissi and others added 24 commits June 26, 2018 01:20
Package-Manager: Portage-2.3.40, Repoman-2.3.9
Package-Manager: Portage-2.3.40, Repoman-2.3.9
Package-Manager: Portage-2.3.40, Repoman-2.3.9
Package-Manager: Portage-2.3.40, Repoman-2.3.9
Package-Manager: Portage-2.3.40, Repoman-2.3.9
Package-Manager: Portage-2.3.40, Repoman-2.3.9
Package-Manager: Portage-2.3.40, Repoman-2.3.9
Package-Manager: Portage-2.3.40, Repoman-2.3.9
Package-Manager: Portage-2.3.40, Repoman-2.3.9
Package-Manager: Portage-2.3.40, Repoman-2.3.9
Package-Manager: Portage-2.3.40, Repoman-2.3.9
Package-Manager: Portage-2.3.40, Repoman-2.3.9
Package-Manager: Portage-2.3.40, Repoman-2.3.9
Package-Manager: Portage-2.3.40, Repoman-2.3.9
Package-Manager: Portage-2.3.40, Repoman-2.3.9
Package-Manager: Portage-2.3.40, Repoman-2.3.9
Package-Manager: Portage-2.3.40, Repoman-2.3.9
Package-Manager: Portage-2.3.40, Repoman-2.3.9
Package-Manager: Portage-2.3.40, Repoman-2.3.9
Package-Manager: Portage-2.3.40, Repoman-2.3.9
Bug: https://bugs.gentoo.org/637956

Making the package -9999 when it isn't actually a live ebuild only
hides the bug from devs. It still affects users trying to emerge the
package because the distfile is still verified against the manifest.

Package-Manager: Portage-2.3.40, Repoman-2.3.9
Closes: https://bugs.gentoo.org/657610
Reported-by: István Haraszti <haraszti.istvan@borsonline.hu>
Package-Manager: Portage-2.3.40, Repoman-2.3.9
Package-Manager: Portage-2.3.40, Repoman-2.3.9
Package-Manager: Portage-2.3.40, Repoman-2.3.9
@gentoo-bot
Copy link

Pull Request assignment

Areas affected: ebuilds
Packages affected: sys-block/targetcli-fb

sys-block/targetcli-fb: @diogocp, @gentoo/proxy-maint

Bugs linked: 657610

In order to force reassignment and/or bug reference scan, please append [please reassign] to the pull request title.

@gentoo-bot gentoo-bot added self-maintained The PR changes only packages that are maintained by the submitter (i.e. no need to ask anybody else) assigned PR successfully assigned to the package maintainer(s). bug linked Bug/Closes found in footer, and cross-linked with the PR. labels Jun 25, 2018
@gentoo-repo-qa-bot
Copy link
Collaborator

Pull request CI report

Report generated at: 2018-06-26 00:34 UTC
Newest commit scanned: 31298de
Status: ✅ good

No issues found

@@ -17,7 +17,7 @@ LICENSE="Apache-2.0"
SLOT="0"
KEYWORDS="~amd64 ~x86"

DEPEND="dev-python/setuptools[${PYTHON_USEDEP}]"
BDEPEND="dev-python/setuptools[${PYTHON_USEDEP}]"
Copy link
Contributor

Choose a reason for hiding this comment

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

DEPEND

Copy link
Contributor Author

Choose a reason for hiding this comment

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

I followed The ultimate guide to EAPI 7:

  1. Dependencies that need to be run during the build, and therefore must run on the system used to build packages (CBUILD). Those include toolchain, build system tooling (autotools, CMake), various language interpreters (Perl, Python), preprocessors (SWIG) and other tools (e.g. pkg-config). Those are placed in BDEPEND now.
  2. Dependencies that need to be compiled for the real system, and present for the toolchain to work. Those mostly include libraries since the link editor needs to link to them. Those remain as DEPEND.

While it doesn't list it explictly, wouldn't setuptools fall under "build system tooling", and thus go into BDEPEND?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
assigned PR successfully assigned to the package maintainer(s). bug linked Bug/Closes found in footer, and cross-linked with the PR. self-maintained The PR changes only packages that are maintained by the submitter (i.e. no need to ask anybody else)
Projects
None yet