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

app-admin/syslog-ng: bump to 3.25.1 (with tests now ;)) #14051

Closed
wants to merge 3 commits into from

Conversation

hydrapolic
Copy link
Contributor

@hydrapolic hydrapolic commented Dec 19, 2019

--- syslog-ng-3.24.1.ebuild     2019-12-18 12:08:50.520675041 +0100
+++ syslog-ng-3.25.1.ebuild     2019-12-19 16:05:56.806475404 +0100
@@ -3,7 +3,7 @@
 
 EAPI=7
 
-PYTHON_COMPAT=( python2_7 python3_{5,6,7,8} )
+PYTHON_COMPAT=( python3_{6,7,8} )
 inherit autotools python-single-r1 systemd
 
 MY_PV_MM=$(ver_cut 1-2)
@@ -14,10 +14,9 @@
 LICENSE="GPL-2+ LGPL-2.1+"
 SLOT="0"
 KEYWORDS="~alpha ~amd64 ~arm ~arm64 ~hppa ~ia64 ~mips ~ppc ~ppc64 ~riscv ~s390 ~sh ~sparc ~x86"
-IUSE="amqp caps dbi geoip2 http ipv6 json kafka libressl mongodb pacct python redis smtp snmp spoof-source systemd tcpd"
+IUSE="amqp caps dbi geoip2 http ipv6 json kafka libressl mongodb pacct python redis smtp snmp test spoof-source systemd tcpd"
 REQUIRED_USE="python? ( ${PYTHON_REQUIRED_USE} )"
-# unit tests require https://github.com/Snaipe/Criterion with additional deps
-RESTRICT="test"
+RESTRICT="!test? ( test )"
 
 RDEPEND="
        >=dev-libs/glib-2.10.1:2
@@ -41,7 +40,8 @@
        tcpd? ( >=sys-apps/tcp-wrappers-7.6 )
        !libressl? ( dev-libs/openssl:0= )
        libressl? ( dev-libs/libressl:0= )"
-DEPEND="${RDEPEND}"
+DEPEND="${RDEPEND}
+       test? ( dev-libs/criterion )"
 BDEPEND="
        sys-devel/flex
        virtual/pkgconfig"

Thanks

Needed for app-admin/syslog-ng tests

Package-Manager: Portage-2.3.82, Repoman-2.3.20
Signed-off-by: Tomáš Mózes <hydrapolic@gmail.com>
Closes: https://bugs.gentoo.org/686276
Package-Manager: Portage-2.3.82, Repoman-2.3.20
Signed-off-by: Tomáš Mózes <hydrapolic@gmail.com>
Signed-off-by: Tomáš Mózes <hydrapolic@gmail.com>
@gentoo-bot
Copy link

Pull Request assignment

Submitter: @hydrapolic
Areas affected: ebuilds, profiles
Packages affected: app-admin/syslog-ng, dev-libs/criterion

app-admin/syslog-ng: @hydrapolic, @gentoo/proxy-maint
dev-libs/criterion: @gentoo/proxy-maint (new package)

Linked bugs

Bugs linked: 686276


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

Docs: Code of ConductCopyright policy (expl.) ● DevmanualGitHub PRsProxy-maint guide

@gentoo-bot gentoo-bot added new package The PR is adding a new package. 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 Dec 19, 2019
@gentoo-repo-qa-bot
Copy link
Collaborator

Pull request CI report

Report generated at: 2019-12-19 16:39 UTC
Newest commit scanned: 03bcbee
Status: ✅ good

Issues already there before the PR (double-check them):
https://qa-reports.gentoo.org/output/gentoo-ci/c4263c2/output.html#app-emulation/virtualbox-additions
https://qa-reports.gentoo.org/output/gentoo-ci/c4263c2/output.html#app-emulation/virtualbox-guest-additions
https://qa-reports.gentoo.org/output/gentoo-ci/c4263c2/output.html#app-emulation/virtualbox-modules
https://qa-reports.gentoo.org/output/gentoo-ci/c4263c2/output.html#dev-db/postgresql
https://qa-reports.gentoo.org/output/gentoo-ci/c4263c2/output.html#dev-java/icedtea
https://qa-reports.gentoo.org/output/gentoo-ci/c4263c2/output.html#dev-php/PEAR-Crypt_CHAP
https://qa-reports.gentoo.org/output/gentoo-ci/c4263c2/output.html#dev-php/PHP_CodeCoverage
https://qa-reports.gentoo.org/output/gentoo-ci/c4263c2/output.html#dev-php/PHP_TokenStream
https://qa-reports.gentoo.org/output/gentoo-ci/c4263c2/output.html#dev-php/phpunit
https://qa-reports.gentoo.org/output/gentoo-ci/c4263c2/output.html#dev-python/parver
https://qa-reports.gentoo.org/output/gentoo-ci/c4263c2/output.html#dev-ros/roslaunch
https://qa-reports.gentoo.org/output/gentoo-ci/c4263c2/output.html#dev-util/cmt
https://qa-reports.gentoo.org/output/gentoo-ci/c4263c2/output.html#gnustep-apps/sogo
https://qa-reports.gentoo.org/output/gentoo-ci/c4263c2/output.html#media-gfx/iscan
https://qa-reports.gentoo.org/output/gentoo-ci/c4263c2/output.html#media-sound/podcatcher
https://qa-reports.gentoo.org/output/gentoo-ci/c4263c2/output.html#media-tv/gentoo-vdr-scripts
https://qa-reports.gentoo.org/output/gentoo-ci/c4263c2/output.html#net-analyzer/nagios-plugins
https://qa-reports.gentoo.org/output/gentoo-ci/c4263c2/output.html#net-analyzer/snort
https://qa-reports.gentoo.org/output/gentoo-ci/c4263c2/output.html#net-analyzer/wireshark
https://qa-reports.gentoo.org/output/gentoo-ci/c4263c2/output.html#net-analyzer/zabbix
https://qa-reports.gentoo.org/output/gentoo-ci/c4263c2/output.html#net-dns/unbound
https://qa-reports.gentoo.org/output/gentoo-ci/c4263c2/output.html#net-libs/mbedtls
https://qa-reports.gentoo.org/output/gentoo-ci/c4263c2/output.html#net-libs/nodejs
https://qa-reports.gentoo.org/output/gentoo-ci/c4263c2/output.html#net-misc/gerbera
https://qa-reports.gentoo.org/output/gentoo-ci/c4263c2/output.html#net-misc/icaclient
https://qa-reports.gentoo.org/output/gentoo-ci/c4263c2/output.html#net-print/cups
https://qa-reports.gentoo.org/output/gentoo-ci/c4263c2/output.html#net-vpn/tor
https://qa-reports.gentoo.org/output/gentoo-ci/c4263c2/output.html#sci-libs/tensorflow
https://qa-reports.gentoo.org/output/gentoo-ci/c4263c2/output.html#sys-apps/policycoreutils
https://qa-reports.gentoo.org/output/gentoo-ci/c4263c2/output.html#sys-auth/nss-pam-ldapd
https://qa-reports.gentoo.org/output/gentoo-ci/c4263c2/output.html#sys-cluster/crmsh
https://qa-reports.gentoo.org/output/gentoo-ci/c4263c2/output.html#sys-cluster/kube-apiserver
https://qa-reports.gentoo.org/output/gentoo-ci/c4263c2/output.html#sys-cluster/kube-controller-manager
https://qa-reports.gentoo.org/output/gentoo-ci/c4263c2/output.html#sys-cluster/kube-scheduler
https://qa-reports.gentoo.org/output/gentoo-ci/c4263c2/output.html#virtual/httpd-php
https://qa-reports.gentoo.org/output/gentoo-ci/c4263c2/output.html#www-apps/netbox
https://qa-reports.gentoo.org/output/gentoo-ci/c4263c2/output.html#www-misc/shellinabox
https://qa-reports.gentoo.org/output/gentoo-ci/c4263c2/output.html#www-servers/varnish
https://qa-reports.gentoo.org/output/gentoo-ci/c4263c2/output.html#x11-libs/fox

Copy link
Member

@juippis juippis left a comment

Choose a reason for hiding this comment

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

Had to restrict tests on criterion because they failed. Gonna file a bug about it, you should fix it before making a KEYWORDING request because arch testers will stop there :)

 * Messages for package dev-libs/criterion-2.3.3:

 * QA Notice: make jobserver unavailable:
 *
 * 	gmake[3]: warning: jobserver unavailable: using -j1.  Add '+' to parent make rule.
 * 	gmake[3]: warning: jobserver unavailable: using -j1.  Add '+' to parent make rule.
 * 	gmake[3]: warning: jobserver unavailable: using -j1.  Add '+' to parent make rule.
 * 	gmake[3]: warning: jobserver unavailable: using -j1.  Add '+' to parent make rule.
 * 	gmake[3]: warning: jobserver unavailable: using -j1.  Add '+' to parent make rule.
 * 	gmake[3]: warning: jobserver unavailable: using -j1.  Add '+' to parent make rule.
 * 	gmake[3]: warning: jobserver unavailable: using -j1.  Add '+' to parent make rule.
 * 	gmake[3]: warning: jobserver unavailable: using -j1.  Add '+' to parent make rule.

Thanks!

@hydrapolic
Copy link
Contributor Author

Thanks @juippis.

Please open a bug for it as tests for criterion/syslog-ng passed on my ~amd64 machine.

That jobserver stuff is probably something to be fixed in the upstream Makefile :-/

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. new package The PR is adding a new package. self-maintained The PR changes only packages that are maintained by the submitter (i.e. no need to ask anybody else)
Projects
None yet
4 participants