[OE-core] Fwd: [PATCH] smart: disable CHANNELSDIR

[Marinescu, Bogdan A]

Sorry, hit "reply" instead of "reply to all" by mistake.

---------- Forwarded message ----------
From: Marinescu, Bogdan A <bogdan.a.marinescu at intel.com>
Date: Tue, Apr 9, 2013 at 11:11 AM
Subject: Re: [OE-core] [PATCH] smart: disable CHANNELSDIR
To: Mark Hatle <mark.hatle at windriver.com>





On Mon, Apr 8, 2013 at 7:05 PM, Mark Hatle <mark.hatle at windriver.com> wrote:

> On 4/8/13 10:02 AM, Bogdan Marinescu wrote:
>
>> Make CHANNELSDIR in smart empty, since this causes host contamination
>> issues
>> on some RPM-based hosts on which smart is already installed.
>>
>> [YOCTO #3881]
>>
>> Signed-off-by: Bogdan Marinescu <bogdan.a.marinescu at intel.com>
>> ---
>>   .../python/python-smartpm/**smart-channelsdir.patch  | 24
>> ++++++++++++++++++++++
>>   .../python/python-smartpm_1.4.**1.bb <http://python-smartpm_1.4.1.bb>                |  3 ++-
>>   2 files changed, 26 insertions(+), 1 deletion(-)
>>   create mode 100644 meta/recipes-devtools/python/**python-smartpm/smart-
>> **channelsdir.patch
>>
>> diff --git a/meta/recipes-devtools/**python/python-smartpm/smart-**channelsdir.patch
>> b/meta/recipes-devtools/**python/python-smartpm/smart-**channelsdir.patch
>> new file mode 100644
>> index 0000000..e621b33
>> --- /dev/null
>> +++ b/meta/recipes-devtools/**python/python-smartpm/smart-**
>> channelsdir.patch
>> @@ -0,0 +1,24 @@
>> +Make CHANNELSDIR in smart empty, since this causes host contamination
>> issues
>> +on some RPM-based hosts on which smart is already installed.
>> +
>> +[YOCTO #3881]
>> +
>> +Upstream-Status: Inappropriate [embedded specific]
>> +
>> +diff --git a/smart/plugins/channelsync.py b/smart/plugins/channelsync.py
>> +index 3ba95ff..646d696 100644
>> +--- a/smart/plugins/channelsync.py
>> ++++ b/smart/plugins/channelsync.py
>> +@@ -23,7 +23,11 @@ from smart.channel import *
>> + from smart import *
>> + import os
>> +
>> +-CHANNELSDIR = "/etc/smart/channels/"
>> ++# For now, we leave the definition of CHANNELSDIR empty. This prevents
>> smart
>> ++# from erroneously consider the  build host's channels while setting up
>> its
>> ++# channels [YOCTO #3881]. If this feature will be used in the future,
>> CHANNELSDIR
>> ++# should be set to a proper value.
>> ++CHANNELSDIR = ""
>>
>
> I don't remember if the channelsdir is used by default on the target or if
> there is a different directory.
>
> Did you check if (on the target) you can still add channels and do a
> remove install/update of a package?


No, I just checked the poky source for any reference to CHANNELSDIR and
didn't find anything. Plus, CHANNELSDIR is set to a fixed directory
(/etc/smart/channels) in the original smart source and I coulnd't find a
similar directory structure in the poky rootfs image. Nevertheless, I'll
test this and let you know the results.

Thanks,
Bogdan


>
>  +
>> + def syncChannels(channelsdir, force=None):
>> +
>> diff --git a/meta/recipes-devtools/**python/python-smartpm_1.4.1.bbb/meta/recipes-devtools/
>> **python/python-smartpm_1.4.1.bb
>> index d92933f..001d9e4 100644
>> --- a/meta/recipes-devtools/**python/python-smartpm_1.4.1.bb
>> +++ b/meta/recipes-devtools/**python/python-smartpm_1.4.1.bb
>> @@ -11,7 +11,7 @@ LICENSE = "GPLv2"
>>   LIC_FILES_CHKSUM = "file://LICENSE;md5=**393a5ca445f6965873eca0259a17f8
>> **33"
>>
>>   DEPENDS = "python rpm"
>> -PR = "r8"
>> +PR = "r9"
>>   SRCNAME = "smart"
>>
>>   SRC_URI = "\
>> @@ -27,6 +27,7 @@ SRC_URI = "\
>>             file://smart-improve-error-**reporting.patch \
>>             file://smart-multilib-fixes.**patch \
>>             file://smart-yaml-error.patch \
>> +          file://smart-channelsdir.patch \
>>             "
>>
>>   SRC_URI[md5sum] = "**573ef32ba177a6b3c4bf7ef04873fc**b6"
>>
>>
>
> ______________________________**_________________
> Openembedded-core mailing list
> Openembedded-core at lists.**openembedded.org<Openembedded-core at lists.openembedded.org>
> http://lists.linuxtogo.org/**cgi-bin/mailman/listinfo/**openembedded-core<http://lists.linuxtogo.org/cgi-bin/mailman/listinfo/openembedded-core>
>
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.openembedded.org/pipermail/openembedded-core/attachments/20130409/7e111017/attachment-0002.html>