[Bug 421231] New: Review Request: kde-l10n - Internationalization support for KDE

[Red Hat Bugzilla]

Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug report.

https://bugzilla.redhat.com/show_bug.cgi?id=421231

Summary: Review Request: kde-l10n - Internationalization support
                    for KDE
           Product: Fedora
           Version: rawhide
          Platform: All
        OS/Version: Linux
            Status: NEW
          Severity: medium
          Priority: medium
         Component: Package Review
        AssignedTo: [e-mail]
        ReportedBy: [e-mail]
         QAContact: [e-mail]
                CC: [e-mail],[e-mail]

Spec URL: http://than.fedorapeople.org/kde-l10n.spec
SRPM URL: kde-l10m srpm is huge, cannot be uploaded on fedorapeople
Description: Internationalization support for KDE

[Bug 421231] Review Request: kde-l10n - Internationalization support for KDE

[Red Hat Bugzilla]

Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug report.

Summary: Review Request: kde-l10n - Internationalization support for KDE

https://bugzilla.redhat.com/show_bug.cgi?id=421231

[e-mail] changed:

What    |Removed                     |Added
----------------------------------------------------------------------------
         AssignedTo|[e-mail]    |[e-mail]
             Status|NEW                         |ASSIGNED

------- Additional Comments From [e-mail]  2007-12-14 13:15 EST -------
I can review this.

[Bug 421231] Review Request: kde-l10n - Internationalization support for KDE

[Red Hat Bugzilla]

Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug report.

Summary: Review Request: kde-l10n - Internationalization support for KDE

https://bugzilla.redhat.com/show_bug.cgi?id=421231

------- Additional Comments From [e-mail]  2007-12-14 13:21 EST -------
Initial suggestions (consider these as SHOULD items):

1. Drop Epoch.  It's a new pkg, shouldn't need it.

2. BuildRequires: kdelibs-devel  ->  BuildRequires: kdelibs4-devel

3. Requires: kdelibs >= 6:%{version} -> Requires: kde-filesystem

4. for each subpkg, add
Provides: %{name}-<locale> = %{version}-%{release}

the latter 3 items similar to recent cvs updates to kde-i18n/devel.

[Bug 421231] Review Request: kde-l10n - Internationalization support for KDE

[Red Hat Bugzilla]

Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug report.

Summary: Review Request: kde-l10n - Internationalization support for KDE

https://bugzilla.redhat.com/show_bug.cgi?id=421231

------- Additional Comments From [e-mail]  2007-12-14 13:59 EST -------
I'd say the Epoch one is a MUST (unless there's a repository out there which 
has the package with the Epoch), we don't want to inflate epochs for no good 
reason. ;-)

[Bug 421231] Review Request: kde-l10n - Internationalization support for KDE

[Red Hat Bugzilla]

Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug report.

Summary: Review Request: kde-l10n - Internationalization support for KDE

https://bugzilla.redhat.com/show_bug.cgi?id=421231

------- Additional Comments From [e-mail]  2007-12-17 06:22 EST -------
the obove issues have been fixed.

[Bug 421231] Review Request: kde-l10n - Internationalization support for KDE

[Red Hat Bugzilla]

Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug report.

Summary: Review Request: kde-l10n - Internationalization support for KDE

https://bugzilla.redhat.com/show_bug.cgi?id=421231

------- Additional Comments From [e-mail]  2007-12-18 06:27 EST -------
Rex, it's great if you could please review the new kde-l10n.spec file again. 
The above issues are now fixed. Thanks

[Bug 421231] Review Request: kde-l10n - Internationalization support for KDE

[Red Hat Bugzilla]

Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug report.

Summary: Review Request: kde-l10n - Internationalization support for KDE

https://bugzilla.redhat.com/show_bug.cgi?id=421231

------- Additional Comments From [e-mail]  2007-12-20 09:25 EST -------
%package Chinese-Big5
should be better renamed to
%package Chinese-Traditional

[Bug 421231] Review Request: kde-l10n - Internationalization support for KDE

[Red Hat Bugzilla]

Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug report.

Summary: Review Request: kde-l10n - Internationalization support for KDE

https://bugzilla.redhat.com/show_bug.cgi?id=421231

------- Additional Comments From [e-mail]  2007-12-20 09:52 EST -------
Wei-Lun, it's now fixed in new kde-l10n.spec. Thanks

[Bug 421231] Review Request: kde-l10n - Internationalization support for KDE

[Red Hat Bugzilla]

Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug report.

Summary: Review Request: kde-l10n - Internationalization support for KDE

https://bugzilla.redhat.com/show_bug.cgi?id=421231

[e-mail] changed:

What    |Removed                     |Added
----------------------------------------------------------------------------
               Flag|                            |fedora-review+

------- Additional Comments From [e-mail]  2007-12-30 23:03 EST -------
Lacking a buildable pkg atm (to be addressed soon, hopefully), everything else
looks good.  APPROVED.

[Bug 421231] Review Request: kde-l10n - Internationalization support for KDE

[Red Hat Bugzilla]

Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug report.

Summary: Review Request: kde-l10n - Internationalization support for KDE

https://bugzilla.redhat.com/show_bug.cgi?id=421231

[e-mail] changed:

What    |Removed                     |Added
----------------------------------------------------------------------------
               Flag|                            |fedora-cvs?

------- Additional Comments From [e-mail]  2008-01-08 02:44 EST -------
New Package CVS Request
=======================
Package Name: kde-l10n
Short Description: Internationalization support for KDE
Owners: than,rdieter,kkofler,ltinkl
Branches:
InitialCC:
Cvsextras Commits: no

[Bug 421231] Review Request: kde-l10n - Internationalization support for KDE

[Red Hat Bugzilla]

Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug report.

Summary: Review Request: kde-l10n - Internationalization support for KDE

https://bugzilla.redhat.com/show_bug.cgi?id=421231

[e-mail] changed:

What    |Removed                     |Added
----------------------------------------------------------------------------
               Flag|fedora-cvs?                 |fedora-cvs+

------- Additional Comments From [e-mail]  2008-01-08 11:44 EST -------
cvs done.

Any particular reason for the cvsextras: no?

[Bug 421231] Review Request: kde-l10n - Internationalization support for KDE

[Red Hat Bugzilla]

Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug report.

Summary: Review Request: kde-l10n - Internationalization support for KDE

https://bugzilla.redhat.com/show_bug.cgi?id=421231

[e-mail] changed:

What    |Removed                     |Added
----------------------------------------------------------------------------
             Status|ASSIGNED                    |CLOSED
         Resolution|                            |RAWHIDE

------- Additional Comments From [e-mail]  2008-02-19 09:59 EST -------
built/included in rawhide now.