[Bug 583058] Review Request: grantlee - Qt string template engine based on the Django template system

2010-06-04 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


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

Fedora Update System upda...@fedoraproject.org changed:

   What|Removed |Added

 Status|ON_QA   |CLOSED
   Fixed In Version||grantlee-0.1.1-3.fc12
 Resolution||ERRATA

-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug.
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review


[Bug 583058] Review Request: grantlee - Qt string template engine based on the Django template system

2010-06-04 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


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

--- Comment #20 from Fedora Update System upda...@fedoraproject.org 
2010-06-04 14:55:59 EDT ---
grantlee-0.1.1-3.fc13 has been pushed to the Fedora 13 stable repository.  If
problems still persist, please make note of it in this bug report.

-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug.
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review


[Bug 583058] Review Request: grantlee - Qt string template engine based on the Django template system

2010-05-19 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


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

Fedora Update System upda...@fedoraproject.org changed:

   What|Removed |Added

 Status|ASSIGNED|ON_QA

--- Comment #17 from Fedora Update System upda...@fedoraproject.org 
2010-05-19 15:11:21 EDT ---
grantlee-0.1.1-3.fc13 has been pushed to the Fedora 13 testing repository.  If
problems still persist, please make note of it in this bug report.
 If you want to test the update, you can install it with 
 su -c 'yum --enablerepo=updates-testing update grantlee'.  You can provide
feedback for this update here:
http://admin.fedoraproject.org/updates/grantlee-0.1.1-3.fc13

-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug.
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review


[Bug 583058] Review Request: grantlee - Qt string template engine based on the Django template system

2010-05-19 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


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

--- Comment #18 from Fedora Update System upda...@fedoraproject.org 
2010-05-19 15:20:13 EDT ---
grantlee-0.1.1-3.fc12 has been pushed to the Fedora 12 testing repository.  If
problems still persist, please make note of it in this bug report.
 If you want to test the update, you can install it with 
 su -c 'yum --enablerepo=updates-testing update grantlee'.  You can provide
feedback for this update here:
http://admin.fedoraproject.org/updates/grantlee-0.1.1-3.fc12

-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug.
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review


[Bug 583058] Review Request: grantlee - Qt string template engine based on the Django template system

2010-05-18 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


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

--- Comment #7 from Jaroslav Reznik jrez...@redhat.com 2010-05-18 11:12:10 
EDT ---
For KDE libraries we agreed on /usr/share/doc/HTML/en/%{name}-apidocs as
kdelibs and kdepimlibs already uses this location and these two are most
important for us.

-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug.
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review


[Bug 583058] Review Request: grantlee - Qt string template engine based on the Django template system

2010-05-18 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


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

--- Comment #8 from Jaroslav Reznik jrez...@redhat.com 2010-05-18 11:21:15 
EDT ---
(In reply to comment #6)
 %{_docdir}/HTML/en/grantlee-apidocs still seems a bit strange, maybe
 %{_docdir}/HTML/grantlee-apidocs/en is more appropriate if we consider the
 fedora-release-notes package.
 
 ls /usr/share/doc/HTML/fedora-release-notes/
 
  cs-CZ  es-ES  index.html  nl-NL  pt-PT  uk-UA
  en-US  fr-FR  it-IT   pl-PL  sv-SE  zh-CN
 
 The installation path and subpackage name for html documentation is really a
 mess in fedora and need further consideration.
 
 The guideline only has one line about this - if there's a lot of 
 documentation,
 consider putting it into a subpackage.
 
 http://fedoraproject.org/wiki/PackagingGuidelines#Documentation
 
 Actually, we now have several subpackage name for html documention.
 E.g
 -doc
 -docs
 -devel-doc
 -devel-docs
 -apidocs
 -manual
 
 
 We also have several installation path for those html files(normally built by
 docbook or doxygen).
 
 %{_docdir}/%{name}-%{version}
 %{_docdir}/%{name}-[subpackagename]-%{version}
 %{_docdir}/%{name}
 %{_docdir}/HTML/%{name}
 %{_docdir}/HTML/[locale]/%{name}
 %{_datadir}/gtk-doc/html/%{name}
 
 In java package guideline,we have
 %{_datadir}/javadoc/%{name}-%{version} for -javadoc subpackage
 %{_docdir}/%{name}-[subpackagename]-%{version} for -manual subpackage
 
 
 I think add a fixed installation path for html apidocs or enduser manuals like
 man or info will benefit all of us a lot .

Indeed it's a mess, so we are trying to fix it at least on our KDE side. But
sistribution wide solution would be really very nice.

-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug.
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review


[Bug 583058] Review Request: grantlee - Qt string template engine based on the Django template system

2010-05-18 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


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

--- Comment #9 from Chen Lei supercyp...@gmail.com 2010-05-18 11:28:43 EDT ---
(In reply to comment #7)
 For KDE libraries we agreed on /usr/share/doc/HTML/en/%{name}-apidocs as
 kdelibs and kdepimlibs already uses this location and these two are most
 important for us.


Can KDE-SIG talk with fedora-release-notes group?

I installed F13 KDE spin, and found a bit inconsistence of using
/usr/share/doc/HTML between kde-filesystem and fedora-release-notes, both
packages are the default components of KDE spin. 

As I said before, fedora-release-notes installs their files to
/usr/share/doc/HTML/fedora-release-notes/, it contains an index.html and some
diretories with different locale names, which conflicts with the using of
/usr/share/doc/HTML in kde-filesystem.

Note:

Currently, in fedora repo kde-filesystem and fedora-release-notes own
/usr/share/doc/HTML.

/usr/share/doc/HTML/en/%{name}-apidocs is the search PATH of kdedevelop.

-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug.
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review


[Bug 583058] Review Request: grantlee - Qt string template engine based on the Django template system

2010-05-18 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


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

Jaroslav Reznik jrez...@redhat.com changed:

   What|Removed |Added

   Flag|fedora-review?  |fedora-review+

--- Comment #11 from Jaroslav Reznik jrez...@redhat.com 2010-05-18 11:46:24 
EDT ---
Thomas, seems like we still don't have proper solution for apidocs - but we
need Grantlee. I'm not going to block this review more - for now, not shipping
apidocs would be best solution (just set apidocs to 0) and add comment
somewhere in SPEC file.

APPROVED

-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug.
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review


[Bug 583058] Review Request: grantlee - Qt string template engine based on the Django template system

2010-05-18 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


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

--- Comment #10 from Jaroslav Reznik jrez...@redhat.com 2010-05-18 11:44:14 
EDT ---
(In reply to comment #9)
 (In reply to comment #7)
  For KDE libraries we agreed on /usr/share/doc/HTML/en/%{name}-apidocs as
  kdelibs and kdepimlibs already uses this location and these two are most
  important for us.
 
 
 Can KDE-SIG talk with fedora-release-notes group?
 
 I installed F13 KDE spin, and found a bit inconsistence of using
 /usr/share/doc/HTML between kde-filesystem and fedora-release-notes, both
 packages are the default components of KDE spin. 
 
 As I said before, fedora-release-notes installs their files to
 /usr/share/doc/HTML/fedora-release-notes/, it contains an index.html and some
 diretories with different locale names, which conflicts with the using of
 /usr/share/doc/HTML in kde-filesystem.
 
 Note:
 
 Currently, in fedora repo kde-filesystem and fedora-release-notes own
 /usr/share/doc/HTML.
 
 /usr/share/doc/HTML/en/%{name}-apidocs is the search PATH of kdedevelop.

It's default KDE upstream installation schema...

-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug.
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review


[Bug 583058] Review Request: grantlee - Qt string template engine based on the Django template system

2010-05-18 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


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

Dennis Gilmore den...@ausil.us changed:

   What|Removed |Added

   Flag|fedora-cvs? |fedora-cvs+

--- Comment #14 from Dennis Gilmore den...@ausil.us 2010-05-18 16:47:10 EDT 
---
CVS Done

-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug.
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review


[Bug 583058] Review Request: grantlee - Qt string template engine based on the Django template system

2010-05-13 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


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

--- Comment #6 from Chen Lei supercyp...@gmail.com 2010-05-13 06:22:55 EDT ---
%{_docdir}/HTML/en/grantlee-apidocs still seems a bit strange, maybe
%{_docdir}/HTML/grantlee-apidocs/en is more appropriate if we consider the
fedora-release-notes package.

ls /usr/share/doc/HTML/fedora-release-notes/

 cs-CZ  es-ES  index.html  nl-NL  pt-PT  uk-UA
 en-US  fr-FR  it-IT   pl-PL  sv-SE  zh-CN

The installation path and subpackage name for html documentation is really a
mess in fedora and need further consideration.

The guideline only has one line about this - if there's a lot of documentation,
consider putting it into a subpackage.

http://fedoraproject.org/wiki/PackagingGuidelines#Documentation

Actually, we now have several subpackage name for html documention.
E.g
-doc
-docs
-devel-doc
-devel-docs
-apidocs
-manual


We also have several installation path for those html files(normally built by
docbook or doxygen).

%{_docdir}/%{name}-%{version}
%{_docdir}/%{name}-[subpackagename]-%{version}
%{_docdir}/%{name}
%{_docdir}/HTML/%{name}
%{_docdir}/HTML/[locale]/%{name}
%{_datadir}/gtk-doc/html/%{name}

In java package guideline,we have
%{_datadir}/javadoc/%{name}-%{version} for -javadoc subpackage
%{_docdir}/%{name}-[subpackagename]-%{version} for -manual subpackage


I think add a fixed installation path for html apidocs or enduser manuals like
man or info will benefit all of us a lot .

-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug.
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review


[Bug 583058] Review Request: grantlee - Qt string template engine based on the Django template system

2010-05-11 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


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

--- Comment #3 from Jaroslav Reznik jrez...@redhat.com 2010-05-11 06:46:36 
EDT ---
Looking on it right now.

For spelling error s/themeing/theming.

-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug.
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review


[Bug 583058] Review Request: grantlee - Qt string template engine based on the Django template system

2010-05-11 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


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

--- Comment #4 from Jaroslav Reznik jrez...@redhat.com 2010-05-11 09:25:38 
EDT ---
Created an attachment (id=413136)
 -- (https://bugzilla.redhat.com/attachment.cgi?id=413136)
Adds apidocs

I'm not sure this patch is correct - location for apidocs?

-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug.
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review


[Bug 583058] Review Request: grantlee - Qt string template engine based on the Django template system

2010-05-09 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


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

--- Comment #2 from Thomas Janssen thom...@fedoraproject.org 2010-05-09 
14:29:37 EDT ---
Spec URL: http://thomasj.fedorapeople.org/reviews/grantlee.spec
SRPM URL: http://thomasj.fedorapeople.org/reviews/grantlee-0.1.1-1.fc12.src.rpm

- fixed Group
- grantlee-0.1.1

You said in IRC you wanted to test the doxygen part out, some progress?

Rpmlint are still some strange spelling warnings, else clean.

http://koji.fedoraproject.org/koji/taskinfo?taskID=2175805

-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug.
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review


[Bug 583058] Review Request: grantlee - Qt string template engine based on the Django template system

2010-04-19 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


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

Jaroslav Reznik jrez...@redhat.com changed:

   What|Removed |Added

 Status|NEW |ASSIGNED
 AssignedTo|nob...@fedoraproject.org|jrez...@redhat.com
   Flag||fedora-review?

--- Comment #1 from Jaroslav Reznik jrez...@redhat.com 2010-04-19 08:15:21 
EDT ---
Rpmlint: ok (spelling warnings)
Package and SPEC name: ok
License: ok
Upstream source code: ok (3ec14214f30cc86d10c3cab2d062ff3e 
grantlee-v0.1.0.tar.gz)
Package builds: ok, see scratch build + tested locally
ldconfig: ok
BuildRequires: ok
Devel split: ok
No libtools archives: ok

Group should be System Environment/Libraries.
I see doxygen and documentation commented out - what's the problem there? I
think it should be shipped together.

-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug.
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review