* [PATCH] python-smartpm: Fix build error @ 2012-12-05 16:13 Saul Wold 2012-12-05 17:42 ` Mark Hatle 0 siblings, 1 reply; 5+ messages in thread From: Saul Wold @ 2012-12-05 16:13 UTC (permalink / raw) To: openembedded-core datadir is normally /usr/share so we don't need to add /share to it. Signed-off-by: Saul Wold <sgw@linux.intel.com> --- meta/recipes-devtools/python/python-smartpm_1.4.1.bb | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/meta/recipes-devtools/python/python-smartpm_1.4.1.bb b/meta/recipes-devtools/python/python-smartpm_1.4.1.bb index 254318c..3617fde 100644 --- a/meta/recipes-devtools/python/python-smartpm_1.4.1.bb +++ b/meta/recipes-devtools/python/python-smartpm_1.4.1.bb @@ -44,7 +44,7 @@ inherit distutils do_install_append() { # Cleanup unused item... - rmdir ${D}${datadir}/share + rmdir ${D}${datadir} # We don't support the following items rm -rf ${D}${libdir}/python*/site-packages/smart/backends/slack -- 1.8.0.1 ^ permalink raw reply related [flat|nested] 5+ messages in thread
* Re: [PATCH] python-smartpm: Fix build error 2012-12-05 16:13 [PATCH] python-smartpm: Fix build error Saul Wold @ 2012-12-05 17:42 ` Mark Hatle 2012-12-05 17:47 ` Martin Jansa 0 siblings, 1 reply; 5+ messages in thread From: Mark Hatle @ 2012-12-05 17:42 UTC (permalink / raw) To: openembedded-core On 12/5/12 10:13 AM, Saul Wold wrote: > datadir is normally /usr/share so we don't need to add /share to it. > > Signed-off-by: Saul Wold <sgw@linux.intel.com> > --- > meta/recipes-devtools/python/python-smartpm_1.4.1.bb | 2 +- > 1 file changed, 1 insertion(+), 1 deletion(-) > > diff --git a/meta/recipes-devtools/python/python-smartpm_1.4.1.bb b/meta/recipes-devtools/python/python-smartpm_1.4.1.bb > index 254318c..3617fde 100644 > --- a/meta/recipes-devtools/python/python-smartpm_1.4.1.bb > +++ b/meta/recipes-devtools/python/python-smartpm_1.4.1.bb > @@ -44,7 +44,7 @@ inherit distutils > > do_install_append() { > # Cleanup unused item... > - rmdir ${D}${datadir}/share > + rmdir ${D}${datadir} I believe this is wrong. There are things in /usr/share that we want from smart. There are docs and locales and associated files... I believe that was there because an errant "/usr/share/share" directory was generated and that caused problems. > > # We don't support the following items > rm -rf ${D}${libdir}/python*/site-packages/smart/backends/slack > ^ permalink raw reply [flat|nested] 5+ messages in thread
* Re: [PATCH] python-smartpm: Fix build error 2012-12-05 17:42 ` Mark Hatle @ 2012-12-05 17:47 ` Martin Jansa 2012-12-05 17:54 ` Mark Hatle 0 siblings, 1 reply; 5+ messages in thread From: Martin Jansa @ 2012-12-05 17:47 UTC (permalink / raw) To: Mark Hatle; +Cc: openembedded-core [-- Attachment #1: Type: text/plain, Size: 1889 bytes --] On Wed, Dec 05, 2012 at 11:42:57AM -0600, Mark Hatle wrote: > On 12/5/12 10:13 AM, Saul Wold wrote: > > datadir is normally /usr/share so we don't need to add /share to it. > > > > Signed-off-by: Saul Wold <sgw@linux.intel.com> > > --- > > meta/recipes-devtools/python/python-smartpm_1.4.1.bb | 2 +- > > 1 file changed, 1 insertion(+), 1 deletion(-) > > > > diff --git a/meta/recipes-devtools/python/python-smartpm_1.4.1.bb b/meta/recipes-devtools/python/python-smartpm_1.4.1.bb > > index 254318c..3617fde 100644 > > --- a/meta/recipes-devtools/python/python-smartpm_1.4.1.bb > > +++ b/meta/recipes-devtools/python/python-smartpm_1.4.1.bb > > @@ -44,7 +44,7 @@ inherit distutils > > > > do_install_append() { > > # Cleanup unused item... > > - rmdir ${D}${datadir}/share > > + rmdir ${D}${datadir} > > I believe this is wrong. There are things in /usr/share that we want from smart. > > There are docs and locales and associated files... > > I believe that was there because an errant "/usr/share/share" directory was > generated and that caused problems. It really was in /usr/share/share and I've moved this fix to distutils.bbclass in http://git.openembedded.org/openembedded-core/commit/?id=10457e343b27e63b35e0278b00e9c25106b0cc1b So rmdir in python-smartpm should be dropped completely... I've fix for that too, but still testing it together with other changes, but will submit that pull-request today. Cheers, > > > > > # We don't support the following items > > rm -rf ${D}${libdir}/python*/site-packages/smart/backends/slack > > > > > _______________________________________________ > Openembedded-core mailing list > Openembedded-core@lists.openembedded.org > http://lists.linuxtogo.org/cgi-bin/mailman/listinfo/openembedded-core -- Martin 'JaMa' Jansa jabber: Martin.Jansa@gmail.com [-- Attachment #2: Digital signature --] [-- Type: application/pgp-signature, Size: 205 bytes --] ^ permalink raw reply [flat|nested] 5+ messages in thread
* Re: [PATCH] python-smartpm: Fix build error 2012-12-05 17:47 ` Martin Jansa @ 2012-12-05 17:54 ` Mark Hatle 2012-12-05 17:57 ` Saul Wold 0 siblings, 1 reply; 5+ messages in thread From: Mark Hatle @ 2012-12-05 17:54 UTC (permalink / raw) To: Martin Jansa; +Cc: openembedded-core On 12/5/12 11:47 AM, Martin Jansa wrote: > On Wed, Dec 05, 2012 at 11:42:57AM -0600, Mark Hatle wrote: >> On 12/5/12 10:13 AM, Saul Wold wrote: >>> datadir is normally /usr/share so we don't need to add /share to it. >>> >>> Signed-off-by: Saul Wold <sgw@linux.intel.com> >>> --- >>> meta/recipes-devtools/python/python-smartpm_1.4.1.bb | 2 +- >>> 1 file changed, 1 insertion(+), 1 deletion(-) >>> >>> diff --git a/meta/recipes-devtools/python/python-smartpm_1.4.1.bb b/meta/recipes-devtools/python/python-smartpm_1.4.1.bb >>> index 254318c..3617fde 100644 >>> --- a/meta/recipes-devtools/python/python-smartpm_1.4.1.bb >>> +++ b/meta/recipes-devtools/python/python-smartpm_1.4.1.bb >>> @@ -44,7 +44,7 @@ inherit distutils >>> >>> do_install_append() { >>> # Cleanup unused item... >>> - rmdir ${D}${datadir}/share >>> + rmdir ${D}${datadir} >> >> I believe this is wrong. There are things in /usr/share that we want from smart. >> >> There are docs and locales and associated files... >> >> I believe that was there because an errant "/usr/share/share" directory was >> generated and that caused problems. > > It really was in /usr/share/share and I've moved this fix to > distutils.bbclass in > http://git.openembedded.org/openembedded-core/commit/?id=10457e343b27e63b35e0278b00e9c25106b0cc1b > > So rmdir in python-smartpm should be dropped completely... I've fix for > that too, but still testing it together with other changes, but will > submit that pull-request today. Ahh, I see.. So yes, the rmdir is simply not needed then and can be removed. --Mark > Cheers, > >> >>> >>> # We don't support the following items >>> rm -rf ${D}${libdir}/python*/site-packages/smart/backends/slack >>> >> >> >> _______________________________________________ >> Openembedded-core mailing list >> Openembedded-core@lists.openembedded.org >> http://lists.linuxtogo.org/cgi-bin/mailman/listinfo/openembedded-core > ^ permalink raw reply [flat|nested] 5+ messages in thread
* Re: [PATCH] python-smartpm: Fix build error 2012-12-05 17:54 ` Mark Hatle @ 2012-12-05 17:57 ` Saul Wold 0 siblings, 0 replies; 5+ messages in thread From: Saul Wold @ 2012-12-05 17:57 UTC (permalink / raw) To: Mark Hatle; +Cc: Martin Jansa, openembedded-core On 12/05/2012 09:54 AM, Mark Hatle wrote: > On 12/5/12 11:47 AM, Martin Jansa wrote: >> On Wed, Dec 05, 2012 at 11:42:57AM -0600, Mark Hatle wrote: >>> On 12/5/12 10:13 AM, Saul Wold wrote: >>>> datadir is normally /usr/share so we don't need to add /share to it. >>>> >>>> Signed-off-by: Saul Wold <sgw@linux.intel.com> >>>> --- >>>> meta/recipes-devtools/python/python-smartpm_1.4.1.bb | 2 +- >>>> 1 file changed, 1 insertion(+), 1 deletion(-) >>>> >>>> diff --git a/meta/recipes-devtools/python/python-smartpm_1.4.1.bb >>>> b/meta/recipes-devtools/python/python-smartpm_1.4.1.bb >>>> index 254318c..3617fde 100644 >>>> --- a/meta/recipes-devtools/python/python-smartpm_1.4.1.bb >>>> +++ b/meta/recipes-devtools/python/python-smartpm_1.4.1.bb >>>> @@ -44,7 +44,7 @@ inherit distutils >>>> >>>> do_install_append() { >>>> # Cleanup unused item... >>>> - rmdir ${D}${datadir}/share >>>> + rmdir ${D}${datadir} >>> >>> I believe this is wrong. There are things in /usr/share that we want >>> from smart. >>> >>> There are docs and locales and associated files... >>> >>> I believe that was there because an errant "/usr/share/share" >>> directory was >>> generated and that caused problems. >> >> It really was in /usr/share/share and I've moved this fix to >> distutils.bbclass in >> http://git.openembedded.org/openembedded-core/commit/?id=10457e343b27e63b35e0278b00e9c25106b0cc1b >> >> >> So rmdir in python-smartpm should be dropped completely... I've fix for >> that too, but still testing it together with other changes, but will >> submit that pull-request today. > > Ahh, I see.. So yes, the rmdir is simply not needed then and can be > removed. > Ok, great, so I guess a patch from Martin shortly will resolved this, I want to get a clean build with MUT for RP to pull this stuff. Thanks Sau! > --Mark > >> Cheers, >> >>> >>>> >>>> # We don't support the following items >>>> rm -rf ${D}${libdir}/python*/site-packages/smart/backends/slack >>>> >>> >>> >>> _______________________________________________ >>> Openembedded-core mailing list >>> Openembedded-core@lists.openembedded.org >>> http://lists.linuxtogo.org/cgi-bin/mailman/listinfo/openembedded-core >> > > > _______________________________________________ > Openembedded-core mailing list > Openembedded-core@lists.openembedded.org > http://lists.linuxtogo.org/cgi-bin/mailman/listinfo/openembedded-core > > ^ permalink raw reply [flat|nested] 5+ messages in thread
end of thread, other threads:[~2012-12-05 18:12 UTC | newest] Thread overview: 5+ messages (download: mbox.gz follow: Atom feed -- links below jump to the message on this page -- 2012-12-05 16:13 [PATCH] python-smartpm: Fix build error Saul Wold 2012-12-05 17:42 ` Mark Hatle 2012-12-05 17:47 ` Martin Jansa 2012-12-05 17:54 ` Mark Hatle 2012-12-05 17:57 ` Saul Wold
This is a public inbox, see mirroring instructions for how to clone and mirror all data and code used for this inbox