[pkg-fso-maint] packaging for qi: please review and comment

[Sebastian Reichel]

On Mon, Jun 06, 2011 at 12:35:56PM +0300, Timo Juhani Lindfors wrote:
> Hi,
> 
> Sebastian Reichel <sre at debian.org> writes:
> > 1. Why do you install to "/usr/lib/qi/gta02" instead of
> >    "/usr/lib/qi/s3c2442"?
> 
> /usr/lib/qi/gta02/qi GTA02 specific. For example it is hardcoded to read
> files named append-GTA02 and uImage-GTA02.bin. -- Wouldn't s3c2442 give
> a bit misleading impression?

According to the qi README it should work on all s3c2442 devices,
that's why I suggested to rename the directory. If that's not the
case /usr/lib/qi/gta02 is better of course. Perhaps the README
should also be updated (Section: "Per-CPU Qi").

> > 2. AFAIK upstream switched from [0] to [1].
> 
> Latest patch in [1] is
> 
> qi-3d7k-more-suspend-gpio-meddle.patch
> 
> and it is already included in the package. There are also 9 patches that
> are not in [1] but are in [0].
> 
> So, is this a fork or new upstream? ;-)

now checking it, I'm not sure there is a real upstream anymore.
There's also [2] from Andy Green (who did >60% of the qi work on
git.openmoko.org).

[0] http://git.openmoko.org/?p=qi.git;a=summary
[1] http://gitorious.org/0xlab-bootloader/qi-bootloader
[2] http://git.warmcat.com/cgi-bin/cgit/qi/

-- Sebastian
-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 836 bytes
Desc: Digital signature
URL: <http://lists.alioth.debian.org/pipermail/pkg-fso-maint/attachments/20110606/a4ce6521/attachment.pgp>