Hi Patrick,
Thank you for the pointing out the note about the python script as well
as the provided details.
On 20.02.2015 12:31, Patrick Ohly wrote:
These bashism issues should indeed be better fixed in the .spec files
and not the .inc files derived from them, so this guide applies.
However, it is uncertain whether working with non-bash shell
interpreters is a goal for Tizen; at the moment it is not tested for
consistently, neither in OBS nor in "Tizen on Yocto". So it is unclear
whether maintainers will accept such patches. Worse, such issues may get
re-introduced.
Overall I think it's easier to accept that .inc files depend on bash and
patch build.py in tizen-distro as suggested above.
As far as I know Ubuntu is among the supported platforms for both Tizen
IVI application and platform development and its default system shell is
dash. In my opinion the change at the python scripts seems as a work
around. It will be more user/developer friendly and more convenient if
the Tizen build with Yocto works out of the box on Ubuntu.
Overall I think it's easier to accept that .inc files depend on bash and
patch build.py in tizen-distro as suggested above.
I saw that one of the proposed by me changes for the Tizen IVI branch
described at JIRA BTY-104 has been already fixed for Tizen:Common by
Ronan Le Martret at commit 92aac3d68d860245be2f59831be1950a07ee6e2b. I
think we should to resolve these bashism issues and I will try to
contribute my changes as soon as the Tizen Gerrit is up and running.
Best regards,
Leon
On 20.02.2015 12:31, Patrick Ohly wrote:
On Fri, 2015-02-20 at 11:52 +0200, Leon Anavi wrote:
Hi,
Recently I have built Tizen IVI with Yocto and I noted several issues
related to the recipes for libmm-common, libmm-log and crosswalk. I have
posted details at JIRA as well as information about the fixes that I
applied locally:
https://bugs.tizen.org/jira/browse/BTY-104
This is the know "bashism" problem mentioned in the release notes for
rev_ivi_2015_02_04 in
https://wiki.tizen.org/wiki/Build_Tizen_with_Yocto_Project#rev_ivi_2015_02_04
* bashisms in recipes derived from .spec files. If you are on a
distro with /bin/sh linking to dash instead of bash,
replace /bin/sh with /bin/bash in
tizen-distro/bitbake/lib/bb/build.py.
I am interested in contributing these fixes upstream. Is the following
guide for contributions up to date and valid for changes related to
Tizen on Yocto?
https://wiki.tizen.org/wiki/How_to_contribute_to_Tizen_IVI
These bashism issues should indeed be better fixed in the .spec files
and not the .inc files derived from them, so this guide applies.
However, it is uncertain whether working with non-bash shell
interpreters is a goal for Tizen; at the moment it is not tested for
consistently, neither in OBS nor in "Tizen on Yocto". So it is unclear
whether maintainers will accept such patches. Worse, such issues may get
re-introduced.
Overall I think it's easier to accept that .inc files depend on bash and
patch build.py in tizen-distro as suggested above.
--
Leon Anavi
Software Engineer
Mob : +359 88 527 7901
konsulko.com
_______________________________________________
IVI mailing list
[email protected]
https://lists.tizen.org/listinfo/ivi
