JPEG library build using RTEMS Graphics Toolkit.

[Gedare Bloom]

On Mon, Oct 6, 2014 at 8:51 PM, Gedare Bloom <gedare at rtems.org> wrote:
> On Mon, Oct 6, 2014 at 7:48 PM, Pavel Pisa <ppisa4lists at pikron.com> wrote:
>> Hello Thomas,
>>
>> On Monday 06 of October 2014 09:52:18 Thomas Kim wrote:
>>> Dear Sir,
>>>
>>> I am tring to build jpeg-8d library using do_it script RTEMS Graphics
>>> Toolkit.
>>>
>>> When I execute "./do_it -j", there is below error message.
>>>
>>> ./do_it: 16: ./do_it: source: not found
>>> VERSION FILE NOT CORRECT
>>
>> which version of RTEMS Graphics Toolkit are you using.
>> I am reproducing with
>>
>> git clone git://git.rtems.org/rtems-graphics-toolkit.git
>> cd rtems-graphics-toolkit
>> git submodule update --init
>>
>> and I can confirm the problem with
>>
>> RTEMS_MAKEFILE_PATH=/opt/rtems4.11/arm-rtems4.11/lpc17xx_ea_ram/ ./do_it -j
>>
>> Reason is that ./do_it scrip is BASH specific and when /bin/sh
>> points to DASH (Debian default) then there is a problem.
>>
>> I suggest next workaround for now
>>
>> diff --git a/do_it b/do_it
>> index 6fe0f86..46aa50f 100755
>> --- a/do_it
>> +++ b/do_it
>> @@ -1,4 +1,4 @@
>> -#! /bin/sh
>> +#! /bin/bash
>>  #
>>  #  Script to help build RTEMS Graphics Toolkit
>>  #
>>
>>
>> The particular problem with reading VERSIONS file should be corrected
>> for DASH by
>>
>> -------------------------------------------
>>
>> diff --git a/do_it b/do_it
>> index 6fe0f86..46aa50f 100755
>> @@ -13,7 +13,7 @@ if [ ! -r ${vfile} ] ; then
>>    exit 1
>>  fi
>>
>> -source ${vfile}
>> +. ./${vfile}
>>
>>  if [ X${LIBJPEG} = X ] ; then
>>    echo VERSION FILE NOT CORRECT
>> @@ -208,7 +208,7 @@ echo "Generating RTEMS_SETTINGS file..."
>>  make -f Makefile.settings clean all
>>  check_status $? Could not generate RTEMS_SETTINGS
>>
>> -source ./RTEMS_SETTINGS
>> +. ./RTEMS_SETTINGS
>>
>>  PREFIX=${BSPTOP}
>>
>> -------------------------------------------
>>
>> Which is correct for BASH as well.
>> Gedare, please, commit it.
>>
> i made all three changes. Maybe it should just be sh and not bash?
> Probably it does not make a big difference. I hope someday someone can
> replace this repository with a set of configs for RSB, but that takes
> some extra effort for building RTEMS libraries there.
>
I made a typo anyway, so I reverted it back to /bin/sh. IF there are
more problems we can address them. I don't know that this script is
really "sh"-safe for all distros.
-Gedare

>> But even with this correction and /bin/sh = DASH there can be another
>> problem. But at least ./do_it -j then works for me.
>>
>> ./do_it -A
>>
>> Has problem that there is missing greation of build directory for Nano-X
>> and even when created manually,
>>
>>   mkdir -p build/microwin/src
>>
>> there is some problem with Microwindows GIT version checkout.
>> We have workaround for that somewhere (Jan Dolezal?) but it should
>> be corrected in Alexandru-Sever's GIT. I have asked for access
>> to that but I am not sure if I have granted access already.
>> The git://github.com/alex-sever-h/microwin.git git should be moved
>> to rtems.org or RTEMS GitHub or integrated as branch to Microwindows
>> mainline as long term solution.
>>
> Yes, we should fix that is a reverse dependency. I'd prefer to see it
> upstream to Microwindows. RTEMS Project should not be carrying around
> ports of 3rd party software unless absolutely necessary.
>
> -Gedare
>
>> Best wishes,
>>
>>              Pavel
>>
>>