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=436753
Summary: Review Request: cdargs - The shell cd with bookmarks and browser Product: Fedora Version: rawhide Platform: All OS/Version: Linux Status: NEW Severity: medium Priority: low Component: Package Review AssignedTo: nobody@fedoraproject.org ReportedBy: xjakub@fi.muni.cz QAContact: extras-qa@fedoraproject.org CC: fedora-package-review@redhat.com,notting@redhat.com
Spec URL: http://www.fi.muni.cz/~xjakub/cdargs/cdargs.spec SRPM URL: http://www.fi.muni.cz/~xjakub/cdargs/cdargs-1.35-1.fc8.src.rpm Description: Enables the user to quickly change working directory by navigating cd arguments and expanding the shell built-in cd with bookmarks and browser.
Koji build: http://koji.fedoraproject.org/koji/taskinfo?taskID=507658
I'm just not sure whether the emacs subpackage is handled well (do not use emacs at all). It is a bit unclear for me from the packaging guidelines for emacs whether there must/should be two subpackages (one for GNU emacs, one for Xemacs) or if it is ok how it is done now (one subpackage for GNU emacs).
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: cdargs - The shell cd with bookmarks and browser
https://bugzilla.redhat.com/show_bug.cgi?id=436753
xjakub@fi.muni.cz changed:
What |Removed |Added ---------------------------------------------------------------------------- OtherBugsDependingO| |177841 nThis| |
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: cdargs - The shell cd with bookmarks and browser
https://bugzilla.redhat.com/show_bug.cgi?id=436753
xjakub@fi.muni.cz changed:
What |Removed |Added ---------------------------------------------------------------------------- OtherBugsDependingO|177841 | nThis| |
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: cdargs - The shell cd with bookmarks and browser
https://bugzilla.redhat.com/show_bug.cgi?id=436753
mmahut@redhat.com changed:
What |Removed |Added ---------------------------------------------------------------------------- AssignedTo|nobody@fedoraproject.org |mmahut@redhat.com Status|NEW |ASSIGNED
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: cdargs - The shell cd with bookmarks and browser Alias: cdargs-review
https://bugzilla.redhat.com/show_bug.cgi?id=436753
mmahut@redhat.com changed:
What |Removed |Added ---------------------------------------------------------------------------- Alias| |cdargs-review Flag| |fedora-review?
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: cdargs - The shell cd with bookmarks and browser Alias: cdargs-review
https://bugzilla.redhat.com/show_bug.cgi?id=436753
mmahut@redhat.com changed:
What |Removed |Added ---------------------------------------------------------------------------- Flag|fedora-review? |fedora-review+
------- Additional Comments From mmahut@redhat.com 2008-03-25 16:12 EST ------- [+] source files match upstream (50be618d67f0b9f2439526193c69c567) [+] latest version is being packaged [-] package meets naming and versioning guidelines - Summary for "emacs-cdargs" does not start with capital letter [+] dist tag is present [+] build root is correct [+] license field matches the actual license (GPLv2+) and is open source-compatible [+] BuildRequires are proper [+] compiler flags are appropriate (using fortify source) [+] %clean is present [+] package builds in mock (dist-f9, dist-f8) [++] package looks sane
Please correct the cosmetic issue with capital letter in summary of emacs-cdargs.
This package is APPROVED.
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: cdargs - The shell cd with bookmarks and browser Alias: cdargs-review
https://bugzilla.redhat.com/show_bug.cgi?id=436753
xjakub@fi.muni.cz changed:
What |Removed |Added ---------------------------------------------------------------------------- Flag| |fedora-cvs?
------- Additional Comments From xjakub@fi.muni.cz 2008-03-25 18:29 EST -------
Please correct the cosmetic issue with capital letter in summary of emacs-cdargs.
Fixed, thanks!
New Spec URL: http://www.fi.muni.cz/~xjakub/cdargs/cdargs.spec New SRPM URL: http://www.fi.muni.cz/~xjakub/cdargs/cdargs-1.35-2.fc8.src.rpm
New Package CVS Request ======================= Package Name: cdargs Short Description: The shell cd with bookmarks and browser Owners: mjakubicek Branches: F-7 F-8 InitialCC: Cvsextras Commits: yes
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: cdargs - The shell cd with bookmarks and browser Alias: cdargs-review
https://bugzilla.redhat.com/show_bug.cgi?id=436753
kevin@tummy.com changed:
What |Removed |Added ---------------------------------------------------------------------------- Flag|fedora-cvs? |fedora-cvs+
------- Additional Comments From kevin@tummy.com 2008-03-26 12:00 EST ------- cvs done.
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: cdargs - The shell cd with bookmarks and browser Alias: cdargs-review
https://bugzilla.redhat.com/show_bug.cgi?id=436753
xjakub@fi.muni.cz changed:
What |Removed |Added ---------------------------------------------------------------------------- Status|ASSIGNED |CLOSED Resolution| |NEXTRELEASE
package-review@lists.fedoraproject.org