---
This is an automatically generated e-mail. To reply, visit:
http://reviewboard.kde.org/r/3228/#review4454
---
Ship it!
Looks good to me. Thanks for taking care!
- Carsten
On
Moin moin
With the commits r1001601, r1001602,r1001603 and r1001608 I moved the
lib to kde-review.
I hope I did everything correctly... I never moved a lib...
There is one CMake-issue left which I am unable to fix: For some reasons
I am getting this issue:
Moin moin
I don't think having such a library in kdelibs is such a bad idea,
actually.
By policy, we usually require at least 2 users of the library before it
can be added into kdelibs. But this seems like something worth
having around in kdelibs to me.
-Allen
On 2009-06-02 08:44:36, Aaron Seigo wrote:
as this introduces new strings (and arguably a new feature :) it will have
to wait for the 4.4 dev cycle to open up.
in the meantime, the i18n here looks very wrong. shouldn't it be:
i18np(density unit, ounce per cubic foot, ounces per
Moin
I'm not sure how to proceed when multiple projects in KDE start to
use the same lib so I let others to answer to that.
they get moved somewhere common, though i don't know where this one
would go. it has kdelibs deps (so kde-support is out), i don't know if
it really fits into kdeibs