Bug 1018501 - Review Request: golang-bitbucket-kardianos-osext - Extensions to the standard Go OS package
Summary: Review Request: golang-bitbucket-kardianos-osext - Extensions to the standard...
Keywords:
Status: CLOSED CURRENTRELEASE
Alias: None
Product: Fedora
Classification: Fedora
Component: Package Review
Version: rawhide
Hardware: All
OS: Linux
medium
medium
Target Milestone: ---
Assignee: Matthew Miller
QA Contact: Fedora Extras Quality Assurance
URL:
Whiteboard:
Depends On:
Blocks: 1018481 1018523
TreeView+ depends on / blocked
 
Reported: 2013-10-12 17:15 UTC by Lokesh Mandvekar
Modified: 2014-07-23 10:04 UTC (History)
9 users (show)

Fixed In Version:
Clone Of:
Environment:
Last Closed: 2013-11-10 18:14:22 UTC
Type: ---
Embargoed:
mattdm: fedora-review+
gwync: fedora-cvs+


Attachments (Terms of Use)

Description Lokesh Mandvekar 2013-10-12 17:15:41 UTC
Spec URL: http://lsm5.fedorapeople.org/rpmbuild/SPECS/golang-bitbucket-kardianos-osext.spec
SRPM URL: http://lsm5.fedorapeople.org/rpmbuild/SRPMS/golang-bitbucket-kardianos-osext-0-0.1.hg364fb577de68.fc21.src.rpm

Description: Provides extra OS functions

Fedora Account System Username: lsm5

Koji: http://koji.fedoraproject.org/koji/taskinfo?taskID=6053727

$ rpmlint SPECS/golang-bitbucket-kardianos-osext.spec SRPMS/golang-bitbucket-kardianos-osext-0-0.1.hg364fb577de68.fc21.src.rpm RPMS/noarch/golang-bitbucket-kardianos-osext-devel-0-0.1.hg364fb577de68.fc21.noarch.rpm
SPECS/golang-bitbucket-kardianos-osext.spec:32: W: setup-not-quiet
golang-bitbucket-kardianos-osext.src:32: W: setup-not-quiet
2 packages and 1 specfiles checked; 0 errors, 2 warnings.

Comment 1 Matthew Miller 2013-10-14 15:02:36 UTC
The summary and description are kind of vague. I suggest

Summary: Extensions to the standard Go os package

%description devel
Provides extensions to the standard Go os package, including Executable, which returns an absolute path which can be used to re-invoke the current program,
and ExecutableFolder, which returns the directory containing the same.

This package contains library source intended for building other packages
which use these functions.

---

Also, I'm not sure if you have this already, but this is a dep of coreos/go-log/log and not etcd proper.

Comment 2 Lokesh Mandvekar 2013-10-14 21:46:39 UTC
(In reply to Matthew Miller from comment #1)
> The summary and description are kind of vague. I suggest
> 
> Summary: Extensions to the standard Go os package
> 
> %description devel
> Provides extensions to the standard Go os package, including Executable,
> which returns an absolute path which can be used to re-invoke the current
> program,
> and ExecutableFolder, which returns the directory containing the same.
> 
> This package contains library source intended for building other packages
> which use these functions.

Done.
> 
> ---
> 
> Also, I'm not sure if you have this already, but this is a dep of
> coreos/go-log/log and not etcd proper.

yup coreos/go-log has been updated with Requires: golang("bitbucket.org/kardianos/osext")

%changelog
* Mon Oct 14 2013 Lokesh Mandvekar <lsm5> 0-0.2.hg364fb577de68
- defattr removed
- description and summary updated

Spec URL: http://lsm5.fedorapeople.org/rpmbuild/SPECS/golang-bitbucket-kardianos-osext.spec
SRPM URL: http://lsm5.fedorapeople.org/rpmbuild/SRPMS/golang-bitbucket-kardianos-osext-0-0.2.hg364fb577de68.fc21.src.rpm

Comment 3 Matthew Miller 2013-10-14 21:55:26 UTC
Review passed, with the same caveat as other Go packages: this may need to be updated as the guidelines evolve beyond draft form.

Comment 4 Lokesh Mandvekar 2013-10-14 22:03:26 UTC
%changelog
* Mon Oct 14 2013 Lokesh Mandvekar <lsm5> 0-0.3.hg364fb577de68
- devel description update

Spec URL: http://lsm5.fedorapeople.org/rpmbuild/SPECS/golang-bitbucket-kardianos-osext.spec
SRPM URL: http://lsm5.fedorapeople.org/rpmbuild/SRPMS/golang-bitbucket-kardianos-osext-0-0.3.hg364fb577de68.fc21.src.rpm

--minor description update

Comment 5 Lokesh Mandvekar 2013-10-14 22:04:32 UTC
New Package SCM Request
=======================
Package Name: golang-bitbucket-kardianos-osext
Short Description: Extensions to the standard Go OS package
Owners: lsm5
Branches: f19 f20 el6
InitialCC:

Comment 6 Jens Petersen 2013-10-15 05:58:15 UTC
Git done (by process-git-requests).

Comment 7 Fedora Update System 2013-10-15 19:35:52 UTC
golang-bitbucket-kardianos-osext-0-0.3.hg364fb577de68.fc20 has been submitted as an update for Fedora 20.
https://admin.fedoraproject.org/updates/golang-bitbucket-kardianos-osext-0-0.3.hg364fb577de68.fc20

Comment 8 Fedora Update System 2013-10-15 19:42:23 UTC
golang-bitbucket-kardianos-osext-0-0.3.hg364fb577de68.fc19 has been submitted as an update for Fedora 19.
https://admin.fedoraproject.org/updates/golang-bitbucket-kardianos-osext-0-0.3.hg364fb577de68.fc19

Comment 9 Fedora Update System 2013-10-15 19:50:07 UTC
golang-bitbucket-kardianos-osext-0-0.3.hg364fb577de68.el6 has been submitted as an update for Fedora EPEL 6.
https://admin.fedoraproject.org/updates/golang-bitbucket-kardianos-osext-0-0.3.hg364fb577de68.el6

Comment 10 Fedora Update System 2013-10-17 18:01:52 UTC
golang-bitbucket-kardianos-osext-0-0.3.hg364fb577de68.el6 has been pushed to the Fedora EPEL 6 testing repository.

Comment 11 Peter Lemenkov 2013-10-23 11:03:20 UTC
Package Change Request
======================
Package Name: golang-bitbucket-kardianos-osext
InitialCC: golang-sig

Comment 12 Gwyn Ciesla 2013-10-23 14:10:12 UTC
Done.

Comment 13 Lokesh Mandvekar 2013-11-10 18:14:22 UTC
i'm closing this since it seems to have made into stable for f20, f19 and el6. please re-open if I got something wrong here. I hope CURRENTRELEASE is the correct flag :)

Comment 14 Troy Dawson 2014-07-22 20:17:55 UTC
Package Change Request
======================
Package Name: golang-bitbucket-kardianos-osext
New Branches: epel7
Owners: tdawson maxamillion

Comment 15 Gwyn Ciesla 2014-07-23 10:04:25 UTC
Git done (by process-git-requests).


Note You need to log in before you can comment on or make changes to this bug.