On Sat, Jan 09, 2010 at 07:19:49PM +0100, Zoff wrote:
> Hi!
> >> whats really crazy is that JaMa's argument was installing 2 packages is 
> >> sooo complicated.
> >
> > WTF? I never said that, maybe you don't understand the difference
> > between having 2 additional recipes and having aditional packages in
> > navit recipe. Separate versioning for those 2 configs is not needed, why
> > I put navit-icons to separate recipe is described bellow.
> you misunderstood.

Then I'm crazy or stupid, but I see
 create mode 100644 recipes/navit/navit-theme-bike_svn.bb
 create mode 100644 recipes/navit/navit-theme-car_svn.bb

in your patch
http://patchwork.dev.bearstech.com/patch/534/

And if you check the status in patchwork its "Changes requested" not
"Refused" or "Stone him!".

> 
> 
> >
> >> but now you have to install navit by hand anyway, and you have to install 
> >> 3 packages now!
> >> navit, navit-locale-*,navit-icons

> again. i dont mean size, i mean if one user wants to install navit now, he 
> has to install 3 pkgs.
> i want only 1 pkg and the depencies will take care of the rest (see my tree 
> below)

No only navit and navit-locale if you don't like default locale,
navit-icons is in RDEPENDS of navit (the same mechanism as you wanted to
use in navit-language-{car|bike}, right?)

Having ${PN}-locale-sth is standard in OE and I guess that users are
quite used to it.

> >> dependencies are:
> >>
> >> navit-austria-car (the icons and some .xml files are in this package)
> >> ->   navit-locale-de_at
> >> ->   navit-icons
> >> ->   navit
> >>       ->   espeak
> >
> > bitb...@jama ~/build.dev.shr.gta $ ls -1
> > ../tmpdir-dev-shr/deploy/ipk/armv4t/navit-locale-* | wc -l
> > 43
> >
> > ah great then we should generate 86 packages and with every revision
> > bump check if navit guys haven't add support for new language.
> no, why should the locales get more?

Someone sends translation to another language to navit guys?

> > And creating something like that would need probably some bbclass
> > change.. don't forget to sent patch for that too
> >
> >> from ./navit/main.c (current svn version!):
> >>           if (getenv("LC_ALL"))
> >>                   dbg(0,"Warning: LC_ALL is set, this might lead to 
> >> problems (e.g. strange positions from GPS)\n");
> >
> > have you tried to add "unset LC_ALL&&" to Exec param in navit.desktop?
> > it works for me
> i also have this in MY desktop file.
> i want it fixed for ALL users , not only for myself.

Why don't you send small git patch for that? It wouldn't be controversal
as generating 86 packages.. I'll commit it if you send it.. Or do it
later (after u-a fix and fso configs recheck) if you won't.., but you
can help us, please do

> > And I still have
> > Settings->Rules->Northing
> > and
> > Settings->Rules->Map follows device

Does this work for you?

> > if it doesn't work, then its not so much SHR devs error.. sorry I cannot
> > test if map follows device with every revision bump.. and yes maybe I'll
> > upgrade revison only locally and never push it to SHR repo..
> 
> ok i give up ...

Me too, sorry if I was rude or something, but please dont blame devs if
they just request some additional changes in patch you sent. They will
be blamed if they push something to upstream branch and someone else
won't like it there, so its better if they feel good about stuff they
are pushing even as only commiters not authors. And shr specific patches
only in shr/merge branch have to die really soon so its not an option
for controversal patches.

Regards,

-- 
uin:136542059                jid:[email protected]
Jansa Martin                 sip:[email protected] 
JaMa                         
_______________________________________________
Shr-devel mailing list
[email protected]
http://lists.shr-project.org/mailman/listinfo/shr-devel

Reply via email to