[OE-core] [PATCH] opkg-utils: allow disabling update-alternatives

2014-08-11 Thread Sujith H 
From: Sujith H sujith_harida...@mentor.com

This is needed to deal with the situation where we're using ipk packaging, so
opkg-utils must be built regardless of what update-alternatives provider we
prefer. The downside to the current implementation is the need to adjust
PACKAGECONFIG as well as PREFERRED_PROVIDER, but it is more explicit that way.

Signed-off-by: Christopher Larson kerg...@gmail.com
Signed-off-by: Sujith H sujith_harida...@mentor.com
---
 meta/recipes-devtools/opkg-utils/opkg-utils_git.bb | 6 +-
 1 file changed, 5 insertions(+), 1 deletion(-)

diff --git a/meta/recipes-devtools/opkg-utils/opkg-utils_git.bb 
b/meta/recipes-devtools/opkg-utils/opkg-utils_git.bb
index 693c216..6fe04ea 100644
--- a/meta/recipes-devtools/opkg-utils/opkg-utils_git.bb
+++ b/meta/recipes-devtools/opkg-utils/opkg-utils_git.bb
@@ -19,11 +19,15 @@ TARGET_CC_ARCH += ${LDFLAGS}
 PYTHONRDEPS = python python-shell python-io python-math python-crypt 
python-logging python-fcntl python-subprocess python-pickle python-compression 
python-textutils python-stringold
 PYTHONRDEPS_class-native = 
 
-PACKAGECONFIG = python
+PACKAGECONFIG = python update-alternatives
 PACKAGECONFIG[python] = ,,,${PYTHONRDEPS}
+PACKAGECONFIG[update-alternatives] = ,,,
 
 do_install() {
oe_runmake PREFIX=${prefix} DESTDIR=${D} install
+if ${@bb.utils.contains('PACKAGECONFIG', 'update-alternatives', 
'false', 'true')}; then
+rm -f ${D}${bindir}/update-alternatives
+fi
 }
 
 PACKAGES =+ update-alternatives-opkg
-- 
1.8.4

-- 
___
Openembedded-core mailing list
Openembedded-core@lists.openembedded.org
http://lists.openembedded.org/mailman/listinfo/openembedded-core

Re: [OE-core] [PATCH] opkg-utils: allow disabling update-alternatives

2014-08-11 Thread sujith h 
On Fri, Aug 8, 2014 at 11:41 PM, Saul Wold s...@linux.intel.com wrote:

 On 08/08/2014 02:37 AM, Sujith H wrote:

 From: Sujith H sujith_harida...@mentor.com

 This is needed to deal with the situation where we're using ipk
 packaging, so
 opkg-utils must be built regardless of what update-alternatives provider
 we
 prefer. The downside to the current implementation is the need to adjust
 PACKAGECONFIG as well as PREFERRED_PROVIDER, but it is more explicit that
 way.

 Signed-off-by: Christopher Larson kerg...@gmail.com
 Signed-off-by: Sujith H sujith_harida...@mentor.com
 ---
   meta/recipes-devtools/opkg-utils/opkg-utils_git.bb | 6 +-
   1 file changed, 5 insertions(+), 1 deletion(-)

 diff --git a/meta/recipes-devtools/opkg-utils/opkg-utils_git.bb
 b/meta/recipes-devtools/opkg-utils/opkg-utils_git.bb
 index 693c216..287bf53 100644
 --- a/meta/recipes-devtools/opkg-utils/opkg-utils_git.bb
 +++ b/meta/recipes-devtools/opkg-utils/opkg-utils_git.bb
 @@ -19,11 +19,15 @@ TARGET_CC_ARCH += ${LDFLAGS}
   PYTHONRDEPS = python python-shell python-io python-math python-crypt
 python-logging python-fcntl python-subprocess python-pickle
 python-compression python-textutils python-stringold
   PYTHONRDEPS_class-native = 

 -PACKAGECONFIG = python
 +PACKAGECONFIG = python update-alternatives
   PACKAGECONFIG[python] = ,,,${PYTHONRDEPS}
 +PACKAGECONFIG[update-alternatives] = ,,,

   do_install() {
 oe_runmake PREFIX=${prefix} DESTDIR=${D} install
 +if ${@'true' if 'update-alternatives' not in
 PACKAGECONFIG.split() else 'false'}; then
 +rm -f ${D}${bindir}/update-alternatives
 +fi

 Would @bb.utils.contains() be better here?  I think that is the correct
 thing to use.


Sure.

 Sau!

}

   PACKAGES =+ update-alternatives-opkg




-- 
സുജിത് ഹരിദാസന്
Bangalore
ProjectContributor to KDE project
http://fci.wikia.com/wiki/Anti-DRM-Campaign
Blog http://sujithh.info
-- 
___
Openembedded-core mailing list
Openembedded-core@lists.openembedded.org
http://lists.openembedded.org/mailman/listinfo/openembedded-core

[OE-core] [PATCH] opkg-utils: allow disabling update-alternatives

2014-08-08 Thread Sujith H 
From: Sujith H sujith_harida...@mentor.com

This is needed to deal with the situation where we're using ipk packaging, so
opkg-utils must be built regardless of what update-alternatives provider we
prefer. The downside to the current implementation is the need to adjust
PACKAGECONFIG as well as PREFERRED_PROVIDER, but it is more explicit that way.

Signed-off-by: Christopher Larson kerg...@gmail.com
Signed-off-by: Sujith H sujith_harida...@mentor.com
---
 meta/recipes-devtools/opkg-utils/opkg-utils_git.bb | 6 +-
 1 file changed, 5 insertions(+), 1 deletion(-)

diff --git a/meta/recipes-devtools/opkg-utils/opkg-utils_git.bb 
b/meta/recipes-devtools/opkg-utils/opkg-utils_git.bb
index 693c216..287bf53 100644
--- a/meta/recipes-devtools/opkg-utils/opkg-utils_git.bb
+++ b/meta/recipes-devtools/opkg-utils/opkg-utils_git.bb
@@ -19,11 +19,15 @@ TARGET_CC_ARCH += ${LDFLAGS}
 PYTHONRDEPS = python python-shell python-io python-math python-crypt 
python-logging python-fcntl python-subprocess python-pickle python-compression 
python-textutils python-stringold
 PYTHONRDEPS_class-native = 
 
-PACKAGECONFIG = python
+PACKAGECONFIG = python update-alternatives
 PACKAGECONFIG[python] = ,,,${PYTHONRDEPS}
+PACKAGECONFIG[update-alternatives] = ,,,
 
 do_install() {
oe_runmake PREFIX=${prefix} DESTDIR=${D} install
+if ${@'true' if 'update-alternatives' not in PACKAGECONFIG.split() 
else 'false'}; then
+rm -f ${D}${bindir}/update-alternatives
+fi
 }
 
 PACKAGES =+ update-alternatives-opkg
-- 
1.8.4

-- 
___
Openembedded-core mailing list
Openembedded-core@lists.openembedded.org
http://lists.openembedded.org/mailman/listinfo/openembedded-core

Re: [OE-core] [PATCH] opkg-utils: allow disabling update-alternatives

2014-08-08 Thread Saul Wold 

On 08/08/2014 02:37 AM, Sujith H wrote:

From: Sujith H sujith_harida...@mentor.com

This is needed to deal with the situation where we're using ipk packaging, so
opkg-utils must be built regardless of what update-alternatives provider we
prefer. The downside to the current implementation is the need to adjust
PACKAGECONFIG as well as PREFERRED_PROVIDER, but it is more explicit that way.

Signed-off-by: Christopher Larson kerg...@gmail.com
Signed-off-by: Sujith H sujith_harida...@mentor.com
---
  meta/recipes-devtools/opkg-utils/opkg-utils_git.bb | 6 +-
  1 file changed, 5 insertions(+), 1 deletion(-)

diff --git a/meta/recipes-devtools/opkg-utils/opkg-utils_git.bb 
b/meta/recipes-devtools/opkg-utils/opkg-utils_git.bb
index 693c216..287bf53 100644
--- a/meta/recipes-devtools/opkg-utils/opkg-utils_git.bb
+++ b/meta/recipes-devtools/opkg-utils/opkg-utils_git.bb
@@ -19,11 +19,15 @@ TARGET_CC_ARCH += ${LDFLAGS}
  PYTHONRDEPS = python python-shell python-io python-math python-crypt 
python-logging python-fcntl python-subprocess python-pickle python-compression 
python-textutils python-stringold
  PYTHONRDEPS_class-native = 

-PACKAGECONFIG = python
+PACKAGECONFIG = python update-alternatives
  PACKAGECONFIG[python] = ,,,${PYTHONRDEPS}
+PACKAGECONFIG[update-alternatives] = ,,,

  do_install() {
oe_runmake PREFIX=${prefix} DESTDIR=${D} install
+if ${@'true' if 'update-alternatives' not in PACKAGECONFIG.split() 
else 'false'}; then
+rm -f ${D}${bindir}/update-alternatives
+fi
Would @bb.utils.contains() be better here?  I think that is the correct 
thing to use.


Sau!


  }

  PACKAGES =+ update-alternatives-opkg


--
___
Openembedded-core mailing list
Openembedded-core@lists.openembedded.org
http://lists.openembedded.org/mailman/listinfo/openembedded-core

[OE-core] [PATCH] opkg-utils: allow disabling update-alternatives

2014-08-07 Thread Sujith H 
From: Sujith H sujith_harida...@mentor.com

This is needed to deal with the situation where we're using ipk packaging, so
opkg-utils must be built regardless of what update-alternatives provider we
prefer. The downside to the current implementation is the need to adjust
PACKAGECONFIG as well as PREFERRED_PROVIDER, but it is more explicit that way.

Signed-off-by: Christopher Larson kerg...@gmail.com
Signed-off-by: Sujith H sujith_harida...@mentor.com
---
 meta/recipes-devtools/opkg-utils/opkg-utils_git.bb | 8 +++-
 1 file changed, 7 insertions(+), 1 deletion(-)

diff --git a/meta/recipes-devtools/opkg-utils/opkg-utils_git.bb 
b/meta/recipes-devtools/opkg-utils/opkg-utils_git.bb
index 693c216..9364740 100644
--- a/meta/recipes-devtools/opkg-utils/opkg-utils_git.bb
+++ b/meta/recipes-devtools/opkg-utils/opkg-utils_git.bb
@@ -19,11 +19,17 @@ TARGET_CC_ARCH += ${LDFLAGS}
 PYTHONRDEPS = python python-shell python-io python-math python-crypt 
python-logging python-fcntl python-subprocess python-pickle python-compression 
python-textutils python-stringold
 PYTHONRDEPS_class-native = 
 
-PACKAGECONFIG = python
+PACKAGECONFIG = python update-alternatives
 PACKAGECONFIG[python] = ,,,${PYTHONRDEPS}
+PACKAGECONFIG[update-alternatives] = ,,,
+
+PROVIDES += ${@'virtual/update-alternatives' if 'update-alternatives' in 
PACKAGECONFIG.split() else ''}
 
 do_install() {
oe_runmake PREFIX=${prefix} DESTDIR=${D} install
+if ${@'true' if 'update-alternatives' not in PACKAGECONFIG.split() 
else 'false'}; then
+rm -f ${D}${bindir}/update-alternatives
+fi
 }
 
 PACKAGES =+ update-alternatives-opkg
-- 
1.8.4

-- 
___
Openembedded-core mailing list
Openembedded-core@lists.openembedded.org
http://lists.openembedded.org/mailman/listinfo/openembedded-core

Re: [OE-core] [PATCH] opkg-utils: allow disabling update-alternatives

2014-08-07 Thread sujith h 
Hi Paul Eggleton,


On Thu, Aug 7, 2014 at 2:54 PM, Sujith H sujit...@gmail.com wrote:

 From: Sujith H sujith_harida...@mentor.com

 This is needed to deal with the situation where we're using ipk packaging,
 so
 opkg-utils must be built regardless of what update-alternatives provider we
 prefer. The downside to the current implementation is the need to adjust
 PACKAGECONFIG as well as PREFERRED_PROVIDER, but it is more explicit that
 way.

 Signed-off-by: Christopher Larson kerg...@gmail.com
 Signed-off-by: Sujith H sujith_harida...@mentor.com
 ---
  meta/recipes-devtools/opkg-utils/opkg-utils_git.bb | 8 +++-
  1 file changed, 7 insertions(+), 1 deletion(-)

 diff --git a/meta/recipes-devtools/opkg-utils/opkg-utils_git.bb
 b/meta/recipes-devtools/opkg-utils/opkg-utils_git.bb
 index 693c216..9364740 100644
 --- a/meta/recipes-devtools/opkg-utils/opkg-utils_git.bb
 +++ b/meta/recipes-devtools/opkg-utils/opkg-utils_git.bb
 @@ -19,11 +19,17 @@ TARGET_CC_ARCH += ${LDFLAGS}
  PYTHONRDEPS = python python-shell python-io python-math python-crypt
 python-logging python-fcntl python-subprocess python-pickle
 python-compression python-textutils python-stringold
  PYTHONRDEPS_class-native = 

 -PACKAGECONFIG = python
 +PACKAGECONFIG = python update-alternatives
  PACKAGECONFIG[python] = ,,,${PYTHONRDEPS}
 +PACKAGECONFIG[update-alternatives] = ,,,
 +
 +PROVIDES += ${@'virtual/update-alternatives' if 'update-alternatives' in
 PACKAGECONFIG.split() else ''}

  do_install() {
 oe_runmake PREFIX=${prefix} DESTDIR=${D} install
 +if ${@'true' if 'update-alternatives' not in
 PACKAGECONFIG.split() else 'false'}; then
 +rm -f ${D}${bindir}/update-alternatives
 +fi
  }

  PACKAGES =+ update-alternatives-opkg
 --
 1.8.4


In this change I am adding it to PROVIDES only if its required. Hope this
change looks ok?

Thanks,
Sujith H

-- 
സുജിത് ഹരിദാസന്
Bangalore
ProjectContributor to KDE project
http://fci.wikia.com/wiki/Anti-DRM-Campaign
Blog http://sujithh.info
-- 
___
Openembedded-core mailing list
Openembedded-core@lists.openembedded.org
http://lists.openembedded.org/mailman/listinfo/openembedded-core

Re: [OE-core] [PATCH] opkg-utils: allow disabling update-alternatives

2014-08-07 Thread Saul Wold 

On 08/07/2014 02:24 AM, Sujith H wrote:

From: Sujith H sujith_harida...@mentor.com

This is needed to deal with the situation where we're using ipk packaging, so
opkg-utils must be built regardless of what update-alternatives provider we
prefer. The downside to the current implementation is the need to adjust
PACKAGECONFIG as well as PREFERRED_PROVIDER, but it is more explicit that way.

Signed-off-by: Christopher Larson kerg...@gmail.com
Signed-off-by: Sujith H sujith_harida...@mentor.com
---
  meta/recipes-devtools/opkg-utils/opkg-utils_git.bb | 8 +++-
  1 file changed, 7 insertions(+), 1 deletion(-)

diff --git a/meta/recipes-devtools/opkg-utils/opkg-utils_git.bb 
b/meta/recipes-devtools/opkg-utils/opkg-utils_git.bb
index 693c216..9364740 100644
--- a/meta/recipes-devtools/opkg-utils/opkg-utils_git.bb
+++ b/meta/recipes-devtools/opkg-utils/opkg-utils_git.bb
@@ -19,11 +19,17 @@ TARGET_CC_ARCH += ${LDFLAGS}
  PYTHONRDEPS = python python-shell python-io python-math python-crypt 
python-logging python-fcntl python-subprocess python-pickle python-compression 
python-textutils python-stringold
  PYTHONRDEPS_class-native = 

-PACKAGECONFIG = python
+PACKAGECONFIG = python update-alternatives
  PACKAGECONFIG[python] = ,,,${PYTHONRDEPS}
+PACKAGECONFIG[update-alternatives] = ,,,
+
+PROVIDES += ${@'virtual/update-alternatives' if 'update-alternatives' in 
PACKAGECONFIG.split() else ''}

Is there not still a PROVIDES line above that has the 
virtual/update-alternative that needs to be removed?


Sau!

  do_install() {
oe_runmake PREFIX=${prefix} DESTDIR=${D} install
+if ${@'true' if 'update-alternatives' not in PACKAGECONFIG.split() 
else 'false'}; then
+rm -f ${D}${bindir}/update-alternatives
+fi
  }

  PACKAGES =+ update-alternatives-opkg


--
___
Openembedded-core mailing list
Openembedded-core@lists.openembedded.org
http://lists.openembedded.org/mailman/listinfo/openembedded-core

[OE-core] [PATCH] opkg-utils: allow disabling update-alternatives

2014-08-06 Thread Sujith H 
From: Sujith H sujith_harida...@mentor.com

This is needed to deal with the situation where we're using ipk packaging, so
opkg-utils must be built regardless of what update-alternatives provider we
prefer. The downside to the current implementation is the need to adjust
PACKAGECONFIG as well as PREFERRED_PROVIDER, but it is more explicit that way.

Signed-off-by: Christopher Larson kerg...@gmail.com
Signed-off-by: Sujith H sujith_harida...@mentor.com
---
 meta/recipes-devtools/opkg-utils/opkg-utils_git.bb | 8 +++-
 1 file changed, 7 insertions(+), 1 deletion(-)

diff --git a/meta/recipes-devtools/opkg-utils/opkg-utils_git.bb 
b/meta/recipes-devtools/opkg-utils/opkg-utils_git.bb
index 693c216..952fce4 100644
--- a/meta/recipes-devtools/opkg-utils/opkg-utils_git.bb
+++ b/meta/recipes-devtools/opkg-utils/opkg-utils_git.bb
@@ -19,11 +19,17 @@ TARGET_CC_ARCH += ${LDFLAGS}
 PYTHONRDEPS = python python-shell python-io python-math python-crypt 
python-logging python-fcntl python-subprocess python-pickle python-compression 
python-textutils python-stringold
 PYTHONRDEPS_class-native = 
 
-PACKAGECONFIG = python
+PACKAGECONFIG = python update-alternatives
 PACKAGECONFIG[python] = ,,,${PYTHONRDEPS}
+PACKAGECONFIG[update-alternatives] = ,,,
+
+PROVIDES_remove = ${@'virtual/update-alternatives' if 'update-alternatives' 
not in PACKAGECONFIG.split() else ''}
 
 do_install() {
oe_runmake PREFIX=${prefix} DESTDIR=${D} install
+if ${@'true' if 'update-alternatives' not in PACKAGECONFIG.split() 
else 'false'}; then
+rm -f ${D}${bindir}/update-alternatives
+fi
 }
 
 PACKAGES =+ update-alternatives-opkg
-- 
1.8.4

-- 
___
Openembedded-core mailing list
Openembedded-core@lists.openembedded.org
http://lists.openembedded.org/mailman/listinfo/openembedded-core

[OE-core] [PATCH] opkg-utils: allow disabling update-alternatives

2014-08-06 Thread Sujith H 
From: Sujith H sujith_harida...@mentor.com

This is needed to deal with the situation where we're using ipk packaging, so
opkg-utils must be built regardless of what update-alternatives provider we
prefer. The downside to the current implementation is the need to adjust
PACKAGECONFIG as well as PREFERRED_PROVIDER, but it is more explicit that way.

Signed-off-by: Christopher Larson kerg...@gmail.com
Signed-off-by: Sujith H sujith_harida...@mentor.com
---
 meta/recipes-devtools/opkg-utils/opkg-utils_git.bb | 8 +++-
 1 file changed, 7 insertions(+), 1 deletion(-)

diff --git a/meta/recipes-devtools/opkg-utils/opkg-utils_git.bb 
b/meta/recipes-devtools/opkg-utils/opkg-utils_git.bb
index 693c216..952fce4 100644
--- a/meta/recipes-devtools/opkg-utils/opkg-utils_git.bb
+++ b/meta/recipes-devtools/opkg-utils/opkg-utils_git.bb
@@ -19,11 +19,17 @@ TARGET_CC_ARCH += ${LDFLAGS}
 PYTHONRDEPS = python python-shell python-io python-math python-crypt 
python-logging python-fcntl python-subprocess python-pickle python-compression 
python-textutils python-stringold
 PYTHONRDEPS_class-native = 
 
-PACKAGECONFIG = python
+PACKAGECONFIG = python update-alternatives
 PACKAGECONFIG[python] = ,,,${PYTHONRDEPS}
+PACKAGECONFIG[update-alternatives] = ,,,
+
+PROVIDES_remove = ${@'virtual/update-alternatives' if 'update-alternatives' 
not in PACKAGECONFIG.split() else ''}
 
 do_install() {
oe_runmake PREFIX=${prefix} DESTDIR=${D} install
+if ${@'true' if 'update-alternatives' not in PACKAGECONFIG.split() 
else 'false'}; then
+rm -f ${D}${bindir}/update-alternatives
+fi
 }
 
 PACKAGES =+ update-alternatives-opkg
-- 
1.8.4

-- 
___
Openembedded-core mailing list
Openembedded-core@lists.openembedded.org
http://lists.openembedded.org/mailman/listinfo/openembedded-core

Re: [OE-core] [PATCH] opkg-utils: allow disabling update-alternatives

2014-08-06 Thread Paul Eggleton 
On Wednesday 06 August 2014 17:34:09 Sujith H wrote:
 From: Sujith H sujith_harida...@mentor.com
 
 This is needed to deal with the situation where we're using ipk packaging,
 so opkg-utils must be built regardless of what update-alternatives provider
 we prefer. The downside to the current implementation is the need to adjust
 PACKAGECONFIG as well as PREFERRED_PROVIDER, but it is more explicit that
 way.
 
 Signed-off-by: Christopher Larson kerg...@gmail.com
 Signed-off-by: Sujith H sujith_harida...@mentor.com
 ---
  meta/recipes-devtools/opkg-utils/opkg-utils_git.bb | 8 +++-
  1 file changed, 7 insertions(+), 1 deletion(-)
 
 diff --git a/meta/recipes-devtools/opkg-utils/opkg-utils_git.bb
 b/meta/recipes-devtools/opkg-utils/opkg-utils_git.bb index 693c216..952fce4
 100644
 --- a/meta/recipes-devtools/opkg-utils/opkg-utils_git.bb
 +++ b/meta/recipes-devtools/opkg-utils/opkg-utils_git.bb
 @@ -19,11 +19,17 @@ TARGET_CC_ARCH += ${LDFLAGS}
  PYTHONRDEPS = python python-shell python-io python-math python-crypt
 python-logging python-fcntl python-subprocess python-pickle
 python-compression python-textutils python-stringold
 PYTHONRDEPS_class-native = 
 
 -PACKAGECONFIG = python
 +PACKAGECONFIG = python update-alternatives
  PACKAGECONFIG[python] = ,,,${PYTHONRDEPS}
 +PACKAGECONFIG[update-alternatives] = ,,,
 +
 +PROVIDES_remove = ${@'virtual/update-alternatives' if
 'update-alternatives' not in PACKAGECONFIG.split() else ''}

If we have to do this, rather than using _remove here could we simply make 
adding it conditional?

Cheers,
Paul

-- 

Paul Eggleton
Intel Open Source Technology Centre
-- 
___
Openembedded-core mailing list
Openembedded-core@lists.openembedded.org
http://lists.openembedded.org/mailman/listinfo/openembedded-core

Re: [OE-core] [PATCH] opkg-utils: allow disabling update-alternatives

2014-08-06 Thread sujith h 
On Wed, Aug 6, 2014 at 6:08 PM, Paul Eggleton paul.eggle...@linux.intel.com
 wrote:

 On Wednesday 06 August 2014 17:34:09 Sujith H wrote:
  From: Sujith H sujith_harida...@mentor.com
 
  This is needed to deal with the situation where we're using ipk
 packaging,
  so opkg-utils must be built regardless of what update-alternatives
 provider
  we prefer. The downside to the current implementation is the need to
 adjust
  PACKAGECONFIG as well as PREFERRED_PROVIDER, but it is more explicit that
  way.
 
  Signed-off-by: Christopher Larson kerg...@gmail.com
  Signed-off-by: Sujith H sujith_harida...@mentor.com
  ---
   meta/recipes-devtools/opkg-utils/opkg-utils_git.bb | 8 +++-
   1 file changed, 7 insertions(+), 1 deletion(-)
 
  diff --git a/meta/recipes-devtools/opkg-utils/opkg-utils_git.bb
  b/meta/recipes-devtools/opkg-utils/opkg-utils_git.bb index
 693c216..952fce4
  100644
  --- a/meta/recipes-devtools/opkg-utils/opkg-utils_git.bb
  +++ b/meta/recipes-devtools/opkg-utils/opkg-utils_git.bb
  @@ -19,11 +19,17 @@ TARGET_CC_ARCH += ${LDFLAGS}
   PYTHONRDEPS = python python-shell python-io python-math python-crypt
  python-logging python-fcntl python-subprocess python-pickle
  python-compression python-textutils python-stringold
  PYTHONRDEPS_class-native = 
 
  -PACKAGECONFIG = python
  +PACKAGECONFIG = python update-alternatives
   PACKAGECONFIG[python] = ,,,${PYTHONRDEPS}
  +PACKAGECONFIG[update-alternatives] = ,,,
  +
  +PROVIDES_remove = ${@'virtual/update-alternatives' if
  'update-alternatives' not in PACKAGECONFIG.split() else ''}

 If we have to do this, rather than using _remove here could we simply make
 adding it conditional?


Isn't _remove itself conditional? Pardon me if my understanding is wrong.



 Cheers,
 Paul

 --

 Paul Eggleton
 Intel Open Source Technology Centre




-- 
സുജിത് ഹരിദാസന്
Bangalore
ProjectContributor to KDE project
http://fci.wikia.com/wiki/Anti-DRM-Campaign
Blog http://sujithh.info
-- 
___
Openembedded-core mailing list
Openembedded-core@lists.openembedded.org
http://lists.openembedded.org/mailman/listinfo/openembedded-core

Re: [OE-core] [PATCH] opkg-utils: allow disabling update-alternatives

2014-08-06 Thread Paul Eggleton 
On Wednesday 06 August 2014 18:58:11 sujith h wrote:
 On Wed, Aug 6, 2014 at 6:08 PM, Paul Eggleton paul.eggle...@linux.intel.com
  wrote:
  
  On Wednesday 06 August 2014 17:34:09 Sujith H wrote:
   From: Sujith H sujith_harida...@mentor.com
   
   This is needed to deal with the situation where we're using ipk
  
  packaging,
  
   so opkg-utils must be built regardless of what update-alternatives
  
  provider
  
   we prefer. The downside to the current implementation is the need to
  
  adjust
  
   PACKAGECONFIG as well as PREFERRED_PROVIDER, but it is more explicit
   that
   way.
   
   Signed-off-by: Christopher Larson kerg...@gmail.com
   Signed-off-by: Sujith H sujith_harida...@mentor.com
   ---
   
meta/recipes-devtools/opkg-utils/opkg-utils_git.bb | 8 +++-
1 file changed, 7 insertions(+), 1 deletion(-)
   
   diff --git a/meta/recipes-devtools/opkg-utils/opkg-utils_git.bb
   b/meta/recipes-devtools/opkg-utils/opkg-utils_git.bb index
  
  693c216..952fce4
  
   100644
   --- a/meta/recipes-devtools/opkg-utils/opkg-utils_git.bb
   +++ b/meta/recipes-devtools/opkg-utils/opkg-utils_git.bb
   @@ -19,11 +19,17 @@ TARGET_CC_ARCH += ${LDFLAGS}
   
PYTHONRDEPS = python python-shell python-io python-math python-crypt
   
   python-logging python-fcntl python-subprocess python-pickle
   python-compression python-textutils python-stringold
   PYTHONRDEPS_class-native = 
   
   -PACKAGECONFIG = python
   +PACKAGECONFIG = python update-alternatives
   
PACKAGECONFIG[python] = ,,,${PYTHONRDEPS}
   
   +PACKAGECONFIG[update-alternatives] = ,,,
   +
   +PROVIDES_remove = ${@'virtual/update-alternatives' if
   'update-alternatives' not in PACKAGECONFIG.split() else ''}
  
  If we have to do this, rather than using _remove here could we simply make
  adding it conditional?
 
 Isn't _remove itself conditional? Pardon me if my understanding is wrong.

What I mean is, this value gets added to PROVIDES here in the same recipe. 
Rather than unconditionally adding it then conditionally removing it, why not 
just conditionally add it?

Cheers,
Paul

-- 

Paul Eggleton
Intel Open Source Technology Centre
-- 
___
Openembedded-core mailing list
Openembedded-core@lists.openembedded.org
http://lists.openembedded.org/mailman/listinfo/openembedded-core