Re: [ptxdist] [PATCH] ptxd_ipkg_rev_smaller: fix compare with empty micro

2011-10-19 Thread Bernhard Walle
* Jon Ringle  [2011-10-19 21:55]:
> >
> What do you think about the bash implementation found here:
> http://lists.us.dell.com/pipermail/dkms-devel/2004-July/000142.html

If it is correct and works, why not.


Regards,
Bernhard

-- 
ptxdist mailing list
ptxdist@pengutronix.de


Re: [ptxdist] [PATCH] ptxd_ipkg_rev_smaller: fix compare with empty micro

2011-10-19 Thread Jon Ringle
On Wed, Oct 19, 2011 at 2:52 PM, Bernhard Walle  wrote:

> Am 19.10.2011 19:46, schrieb Robert Schwebel:
>
>  On Tue, Oct 18, 2011 at 04:42:46AM -0400, Jon Ringle wrote:
>>
>>> After more testing, I found that there were other situations where the
>>> version
>>> comparison does not work well.
>>>
>>> I ended up replacing the implementation of ptxd_ipkg_rev_smaller() with
>>> something much simpler, albeit dependent on dpkg that seems to work well:
>>>
>>> ptxd_ipkg_rev_smaller() {
>>> dpkg --compare-versions $1 lt $2
>>> }
>>>
>>
>> Hmm, that makes us dependend on dpkg (and thus on Debian). PTXdist tries
>> hard to run on any distribution, so I'm not happy with this.
>>
>
> Maybe Perl would be okay?
>
> http://www.emdebian.org/trac/**browser/current/target/dpkg/**
> trunk/scripts/Dpkg/Version.pm?**rev=6998
>
> Seems to be quite official, and the Dpkg::ErrorHandling and Dpkg::Gettext
> could be easily removed.
>
>
What do you think about the bash implementation found here:
http://lists.us.dell.com/pipermail/dkms-devel/2004-July/000142.html

Jon
-- 
ptxdist mailing list
ptxdist@pengutronix.de


Re: [ptxdist] [PATCH] ptxd_ipkg_rev_smaller: fix compare with empty micro

2011-10-19 Thread Bernhard Walle

Am 19.10.2011 19:46, schrieb Robert Schwebel:

On Tue, Oct 18, 2011 at 04:42:46AM -0400, Jon Ringle wrote:

After more testing, I found that there were other situations where the version
comparison does not work well.

I ended up replacing the implementation of ptxd_ipkg_rev_smaller() with
something much simpler, albeit dependent on dpkg that seems to work well:

ptxd_ipkg_rev_smaller() {
 dpkg --compare-versions $1 lt $2
}


Hmm, that makes us dependend on dpkg (and thus on Debian). PTXdist tries
hard to run on any distribution, so I'm not happy with this.


Maybe Perl would be okay?

http://www.emdebian.org/trac/browser/current/target/dpkg/trunk/scripts/Dpkg/Version.pm?rev=6998

Seems to be quite official, and the Dpkg::ErrorHandling and 
Dpkg::Gettext could be easily removed.



Regards,
Bernhard

--
ptxdist mailing list
ptxdist@pengutronix.de


Re: [ptxdist] [PATCH] ptxd_ipkg_rev_smaller: fix compare with empty micro

2011-10-19 Thread Robert Schwebel
On Tue, Oct 18, 2011 at 04:42:46AM -0400, Jon Ringle wrote:
> After more testing, I found that there were other situations where the version
> comparison does not work well.
> 
> I ended up replacing the implementation of ptxd_ipkg_rev_smaller() with
> something much simpler, albeit dependent on dpkg that seems to work well:
> 
> ptxd_ipkg_rev_smaller() {
>     dpkg --compare-versions $1 lt $2
> }

Hmm, that makes us dependend on dpkg (and thus on Debian). PTXdist tries
hard to run on any distribution, so I'm not happy with this.

rsc
-- 
Pengutronix e.K.   | |
Industrial Linux Solutions | http://www.pengutronix.de/  |
Peiner Str. 6-8, 31137 Hildesheim, Germany | Phone: +49-5121-206917-0|
Amtsgericht Hildesheim, HRA 2686   | Fax:   +49-5121-206917- |

-- 
ptxdist mailing list
ptxdist@pengutronix.de


Re: [ptxdist] [oselas] Fail to build the OSELAS.Toolchain

2011-10-19 Thread Juergen Beisert
Hi George,

Juergen Beisert wrote:
> once again: Please keep the mailing list at CC.
>
> George Ng wrote:
> > The problem is the ptxdixt can not be executed.
> >
> > even when ptxdist --version
> >
> > I got
> >
> > ptxdist: error: cannot create logdir: '/local/platform'

Did you run this command in the "/local" dir as the current working directory?

[jb@isonoe]/usr/local/platform > ptxdist-2011.03.1 --version
ptxdist: error: cannot create logdir: '/usr/local/platform'

[jb@isonoe]/usr/local/platform > cd ~
[jb@isonoe]~ > ptxdist-2011.03.1 --version
2011.03.1

And now I see your issue:

[GeorgeNotebook@GeorgeNotebook opt]$ cd OSELAS.Toolchain-2011.03.1
[GeorgeNotebook@GeorgeNotebook OSELAS.Toolchain-2011.03.1]$ ptxdist select
ptxconfigs/
ptxdist: error: cannot create logdir:
'/opt/OSELAS.Toolchain-2011.03.1/platform'

It seems you run these command in "/opt", didn't you?

Building the OSELAS.Toolchain must be done as a regular user and at a location 
where you have write permission (for example in your home directory).
After building the toolchain only the results are installed to "opt/".

Regards,
Juergen

-- 
Pengutronix e.K.  | Juergen Beisert |
Linux Solutions for Science and Industry  | http://www.pengutronix.de/  |

-- 
ptxdist mailing list
ptxdist@pengutronix.de


Re: [ptxdist] [oselas] Fail to build the OSELAS.Toolchain

2011-10-19 Thread Juergen Beisert
Hi George,

once again: Please keep the mailing list at CC.

George Ng wrote:
> The problem is the ptxdixt can not be executed.
>
> even when ptxdist --version
>
> I got
>
> ptxdist: error: cannot create logdir: '/local/platform'
>
> I think I have to re-install the ptxdist-2011.08.0.tar.bz2 again.
>
> Can you tell me how to remove the one currently installed in my host?

If you do not change any setting with '--prefix' in the 'configure' step, 
PTXdist should be installed to "/usr/local/lib/ptxdist-/" 
and "/usr/local/bin/ptxdist-". De-installing can be done by 
removing these files and directories.

On the other hand, you may could start with removing the PTXdist's rc file 
in "~/.ptxdist/" and test again.

Regards,
Juergen

-- 
Pengutronix e.K.  | Juergen Beisert |
Linux Solutions for Science and Industry  | http://www.pengutronix.de/  |

-- 
ptxdist mailing list
ptxdist@pengutronix.de