[Bug 260441] Review Request: pguiman - The PostgreSQL database server managing tool
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: pguiman - The PostgreSQL database server managing tool https://bugzilla.redhat.com/show_bug.cgi?id=260441 [EMAIL PROTECTED] changed: What|Removed |Added Status|ASSIGNED|CLOSED Resolution||NEXTRELEASE -- 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, or are watching someone who is. ___ Fedora-package-review mailing list Fedora-package-review@redhat.com http://www.redhat.com/mailman/listinfo/fedora-package-review
[Bug 260441] Review Request: pguiman - The PostgreSQL database server managing tool
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: pguiman - The PostgreSQL database server managing tool https://bugzilla.redhat.com/show_bug.cgi?id=260441 [EMAIL PROTECTED] changed: What|Removed |Added Flag|fedora-cvs+ |fedora-cvs? --- Additional Comments From [EMAIL PROTECTED] 2007-09-14 05:41 EST --- a cvs admin request from me, [EMAIL PROTECTED], Fedora login kasal: 1) The PackageDB contains packages odvorace and jbrassow which are obviously bogus. Would you be so kind to delete them? (Or could you please fwd the request to the authorities, if this CVS Admin request it not the right one? (I searched the wiki, but was not able to find anything better.)) 2) Perhaps a similar mistake/bug has happened in other cases as well. Could you perhaps query the db for pkgs with names == some login? Or query for pkgs which have only the name set, without any details given? Thanks! -- 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, or are watching someone who is. ___ Fedora-package-review mailing list Fedora-package-review@redhat.com http://www.redhat.com/mailman/listinfo/fedora-package-review
[Bug 260441] Review Request: pguiman - The PostgreSQL database server managing tool
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: pguiman - The PostgreSQL database server managing tool https://bugzilla.redhat.com/show_bug.cgi?id=260441 [EMAIL PROTECTED] changed: What|Removed |Added Flag|fedora-cvs? |fedora-cvs+ --- Additional Comments From [EMAIL PROTECTED] 2007-09-14 15:01 EST --- Thanks for catching this! I've marked the packages removed and am updating the UI code to ignore their existence. Scanned the DB for packages which had a similar lack of other details and found none. -- 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, or are watching someone who is. ___ Fedora-package-review mailing list Fedora-package-review@redhat.com http://www.redhat.com/mailman/listinfo/fedora-package-review
[Bug 260441] Review Request: pguiman - The PostgreSQL database server managing tool
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: pguiman - The PostgreSQL database server managing tool https://bugzilla.redhat.com/show_bug.cgi?id=260441 [EMAIL PROTECTED] changed: What|Removed |Added Flag||fedora-cvs? --- Additional Comments From [EMAIL PROTECTED] 2007-09-12 11:22 EST --- Ondrej, please don't forget to set fedora-cvs flag to ? . -- 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, or are watching someone who is. ___ Fedora-package-review mailing list Fedora-package-review@redhat.com http://www.redhat.com/mailman/listinfo/fedora-package-review
[Bug 260441] Review Request: pguiman - The PostgreSQL database server managing tool
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: pguiman - The PostgreSQL database server managing tool https://bugzilla.redhat.com/show_bug.cgi?id=260441 [EMAIL PROTECTED] changed: What|Removed |Added Flag|fedora-cvs? |fedora-cvs+ --- Additional Comments From [EMAIL PROTECTED] 2007-09-12 12:31 EST --- 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, or are watching someone who is. ___ Fedora-package-review mailing list Fedora-package-review@redhat.com http://www.redhat.com/mailman/listinfo/fedora-package-review
[Bug 260441] Review Request: pguiman - The PostgreSQL database server managing tool
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: pguiman - The PostgreSQL database server managing tool https://bugzilla.redhat.com/show_bug.cgi?id=260441 [EMAIL PROTECTED] changed: What|Removed |Added AssignedTo|[EMAIL PROTECTED] |[EMAIL PROTECTED] Flag|fedora-review? |fedora-review+ -- 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, or are watching someone who is. ___ Fedora-package-review mailing list Fedora-package-review@redhat.com http://www.redhat.com/mailman/listinfo/fedora-package-review
[Bug 260441] Review Request: pguiman - The PostgreSQL database server managing tool
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: pguiman - The PostgreSQL database server managing tool https://bugzilla.redhat.com/show_bug.cgi?id=260441 [EMAIL PROTECTED] changed: What|Removed |Added AssignedTo|[EMAIL PROTECTED] |[EMAIL PROTECTED] --- Additional Comments From [EMAIL PROTECTED] 2007-09-07 04:24 EST --- ( Well, at some time there was a suggestion that the assignee should be back to the submitter after the review is accepted, however now it is not. Please leave the assignee as it is even after the review is accepted ) -- 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, or are watching someone who is. ___ Fedora-package-review mailing list Fedora-package-review@redhat.com http://www.redhat.com/mailman/listinfo/fedora-package-review
[Bug 260441] Review Request: pguiman - The PostgreSQL database server managing tool
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: pguiman - The PostgreSQL database server managing tool https://bugzilla.redhat.com/show_bug.cgi?id=260441 --- Additional Comments From [EMAIL PROTECTED] 2007-09-07 03:57 EST --- New Package CVS Request === Package Name: pguiman Short Description: The PostgreSQL database server managing tool Owners: odvorace Branches: FC-6 F-7 InitialCC: Cvsextras Commits: no -- 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, or are watching someone who is. ___ Fedora-package-review mailing list Fedora-package-review@redhat.com http://www.redhat.com/mailman/listinfo/fedora-package-review
[Bug 260441] Review Request: pguiman - The PostgreSQL database server managing tool
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: pguiman - The PostgreSQL database server managing tool https://bugzilla.redhat.com/show_bug.cgi?id=260441 [EMAIL PROTECTED] changed: What|Removed |Added AssignedTo|[EMAIL PROTECTED]|[EMAIL PROTECTED] -- 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, or are watching someone who is. ___ Fedora-package-review mailing list Fedora-package-review@redhat.com http://www.redhat.com/mailman/listinfo/fedora-package-review
[Bug 260441] Review Request: pguiman - The PostgreSQL database server managing tool
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: pguiman - The PostgreSQL database server managing tool https://bugzilla.redhat.com/show_bug.cgi?id=260441 [EMAIL PROTECTED] changed: What|Removed |Added Status|NEW |ASSIGNED Flag||fedora-review? -- 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, or are watching someone who is. ___ Fedora-package-review mailing list Fedora-package-review@redhat.com http://www.redhat.com/mailman/listinfo/fedora-package-review
[Bug 260441] Review Request: pguiman - The PostgreSQL database server managing tool
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: pguiman - The PostgreSQL database server managing tool
https://bugzilla.redhat.com/show_bug.cgi?id=260441
--- Additional Comments From [EMAIL PROTECTED] 2007-09-06 08:42 EST ---
* RPM name is OK
* Builds fine in mock
* File list looks OK
X Missing SMP flags. If it doesn't build with it, please add a comment
(wiki: PackagingGuidelines#parallelmake)
X rpmlint is not silent, see below
* dist tag is present.
* license field matches the actual license.
* %clean is present.
X sources are listed twice
- Please remove Source0
* spec file for the package is legible
* package builds properly
* package installs properly
? package doesn't use locales
* no duplicates in %files.
* file permissions are appropriate.
* no -devel package
* no -docs subpackage is necessary
* no pkgconfig files
* good use of macros
* content is permissable
* %doc has no affect on runtime
* no static libraries present
X package owns directories already owned by aother package
- Please remove %dir %{_datadir}/applications/ and %{_datadir}/pixmaps/
* package contains %{name}.desktop file, desktop-file-install command present
SHOULD:
X package doesn't contains LICENSE file
* no Non-English languages in description or summary
* build tested on mock-build x86_64 and i386
* tested functionaloty of package
rpmlint:
W: pguiman-debuginfo spurious-executable-perm
/usr/src/debug/pguiman-0.0.1/src/database/create_db_objects.c
W: pguiman-debuginfo spurious-executable-perm
/usr/src/debug/pguiman-0.0.1/src/tools/db_profile_manager.c
W: pguiman-debuginfo spurious-executable-perm
/usr/src/debug/pguiman-0.0.1/src/GUI_db_objects/domain.c
...
Simple solution: remove execution bit from all files *.c *.h
--
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, or are watching someone who is.
___
Fedora-package-review mailing list
Fedora-package-review@redhat.com
http://www.redhat.com/mailman/listinfo/fedora-package-review
[Bug 260441] Review Request: pguiman - The PostgreSQL database server managing tool
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: pguiman - The PostgreSQL database server managing tool
https://bugzilla.redhat.com/show_bug.cgi?id=260441
--- Additional Comments From [EMAIL PROTECTED] 2007-09-06 11:18 EST ---
Hi,
links to corrected spec file and srpm:
Spec URL:
http://pguiman.svn.sourceforge.net/viewvc/*checkout*/pguiman/trunk/pguiman.spec
SRPM URL: http://downloads.sourceforge.net/pguiman/pguiman-0.0.1-2.fc7.src.rpm
Comments:
MUST:
X Missing SMP flags. If it doesn't build with it, please add a comment
(wiki: PackagingGuidelines#parallelmake)
## added SMP flags
X sources are listed twice
- Please remove Source0
## left only sourceforge.net source
X package owns directories already owned by aother package
- Please remove %dir %{_datadir}/applications/ and %{_datadir}/pixmaps/
## removed %dir %{_datadir}/applications/ and %dir %{_datadir}/pixmaps/ from
spec file
SHOULD:
X package doesn't contains LICENSE file
## the licence text is in the COPYING file
rpmlint:
W: pguiman-debuginfo spurious-executable-perm
/usr/src/debug/pguiman-0.0.1/src/database/create_db_objects.c
W: pguiman-debuginfo spurious-executable-perm
/usr/src/debug/pguiman-0.0.1/src/tools/db_profile_manager.c
W: pguiman-debuginfo spurious-executable-perm
/usr/src/debug/pguiman-0.0.1/src/GUI_db_objects/domain.c
...
Simple solution: remove execution bit from all files *.c *.h
## removed execution bit from all files *.c *h
--
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, or are watching someone who is.
___
Fedora-package-review mailing list
Fedora-package-review@redhat.com
http://www.redhat.com/mailman/listinfo/fedora-package-review
