[PATCH 1/2] phoronix-test-suite: add rconflict

[PATCH 1/2] phoronix-test-suite: add rconflict

[changqing.li]

From: Changqing Li <changqing.li@windriver.com>

phoronix-test-suite rdepend on lsb, lsb rdepend on lsbinitscripts,
lsbinitscripts rconflict with initscripts-functions. so also add
rconflict for this one.

Signed-off-by: Changqing Li <changqing.li@windriver.com>
---
 .../recipes-benchmark/phoronix-test-suite/phoronix-test-suite_8.6.0.bb   | 1 +
 1 file changed, 1 insertion(+)

diff --git a/meta-oe/recipes-benchmark/phoronix-test-suite/phoronix-test-suite_8.6.0.bb b/meta-oe/recipes-benchmark/phoronix-test-suite/phoronix-test-suite_8.6.0.bb
index 4c7b057..bdd6855 100644
--- a/meta-oe/recipes-benchmark/phoronix-test-suite/phoronix-test-suite_8.6.0.bb
+++ b/meta-oe/recipes-benchmark/phoronix-test-suite/phoronix-test-suite_8.6.0.bb
@@ -30,6 +30,7 @@ SYSTEMD_AUTO_ENABLE = "disable"
 SYSTEMD_SERVICE_${PN} = "phoromatic-client.service phoromatic-server.service"
 
 RDEPENDS_${PN} += "bash python php-cli util-linux-lscpu os-release lsb"
+RCONFLICTS_${PN} = "initscripts-functions"
 
 FILES_${PN} += " \
     ${datadir}/phoronix-test-suite \
-- 
2.7.4

[PATCH 2/2] syslog-ng: add rconflict for package syslog-ng-libs

[changqing.li]

From: Changqing Li <changqing.li@windriver.com>

syslog-ng-libs also have conflicts

Signed-off-by: Changqing Li <changqing.li@windriver.com>
---
 meta-oe/recipes-support/syslog-ng/syslog-ng.inc | 1 +
 1 file changed, 1 insertion(+)

diff --git a/meta-oe/recipes-support/syslog-ng/syslog-ng.inc b/meta-oe/recipes-support/syslog-ng/syslog-ng.inc
index b52deff..c183f05 100644
--- a/meta-oe/recipes-support/syslog-ng/syslog-ng.inc
+++ b/meta-oe/recipes-support/syslog-ng/syslog-ng.inc
@@ -127,6 +127,7 @@ CONFFILES_${PN} = "${sysconfdir}/${BPN}.conf ${sysconfdir}/scl.conf"
 RDEPENDS_${PN} += " ${@oe.utils.conditional("ONLINE_PACKAGE_MANAGEMENT", "none", "", "update-rc.d", d)}"
 
 RCONFLICTS_${PN} = "busybox-syslog sysklogd rsyslog"
+RCONFLICTS_${PN}-libs = "busybox-syslog sysklogd rsyslog"
 
 RPROVIDES_${PN} += "${PN}-systemd"
 RREPLACES_${PN} += "${PN}-systemd"
-- 
2.7.4

Re: [PATCH 1/2] phoronix-test-suite: add rconflict

[Adrian Bunk]

On Tue, May 28, 2019 at 10:51:58AM +0800, changqing.li@windriver.com wrote:
> From: Changqing Li <changqing.li@windriver.com>
> 
> phoronix-test-suite rdepend on lsb, lsb rdepend on lsbinitscripts,
> lsbinitscripts rconflict with initscripts-functions. so also add
> rconflict for this one.
>...

In what situation is this needed?

It sounds wrong if rconflict from lower-level packages have to be 
replicated in higher-level packages, and is likely to break in the
future.

cu
Adrian

-- 

       "Is there not promise of rain?" Ling Tan asked suddenly out
        of the darkness. There had been need of rain for many days.
       "Only a promise," Lao Er said.
                                       Pearl S. Buck - Dragon Seed

Re: [PATCH 1/2] phoronix-test-suite: add rconflict

[Changqing Li]

On 5/29/19 3:37 AM, Adrian Bunk wrote:
> On Tue, May 28, 2019 at 10:51:58AM +0800, changqing.li@windriver.com wrote:
>> From: Changqing Li <changqing.li@windriver.com>
>>
>> phoronix-test-suite rdepend on lsb, lsb rdepend on lsbinitscripts,
>> lsbinitscripts rconflict with initscripts-functions. so also add
>> rconflict for this one.
>> ...
> In what situation is this needed?
>
> It sounds wrong if rconflict from lower-level packages have to be
> replicated in higher-level packages, and is likely to break in the
> future.
>
> cu
> Adrian

My original idea is that,  by add this info,  we can get the rconflict  
info when parse recipe.

But, you are right,  this  is not good for an indirect rconflicts.


@Khem,  I noticed this is on master-next,  so maybe this patch 1/2 
needed to be reverted.  Sorry for the inconvenience.

-- 
BRs

Sandy(Li Changqing)

Re: [PATCH 1/2] phoronix-test-suite: add rconflict

[Khem Raj]

On Wed, May 29, 2019 at 4:51 AM Changqing Li <changqing.li@windriver.com> wrote:
>
> On 5/29/19 3:37 AM, Adrian Bunk wrote:
> > On Tue, May 28, 2019 at 10:51:58AM +0800, changqing.li@windriver.com wrote:
> >> From: Changqing Li <changqing.li@windriver.com>
> >>
> >> phoronix-test-suite rdepend on lsb, lsb rdepend on lsbinitscripts,
> >> lsbinitscripts rconflict with initscripts-functions. so also add
> >> rconflict for this one.
> >> ...
> > In what situation is this needed?
> >
> > It sounds wrong if rconflict from lower-level packages have to be
> > replicated in higher-level packages, and is likely to break in the
> > future.
> >
> > cu
> > Adrian
>
> My original idea is that,  by add this info,  we can get the rconflict
> info when parse recipe.
>
> But, you are right,  this  is not good for an indirect rconflicts.
>
>
> @Khem,  I noticed this is on master-next,  so maybe this patch 1/2
> needed to be reverted.  Sorry for the inconvenience.
>

thanks for letting me know, I have dropped it.
> --
> BRs
>
> Sandy(Li Changqing)
>