[Bug 1336168] Review Request: git-lfs - Git extension for versioning large files

[bugzilla]

https://bugzilla.redhat.com/show_bug.cgi?id=1336168
Bug 1336168 depends on bug 1486510, which changed state.

Bug 1486510 Summary: Review Request: golang-github-bgentry-go-netrc - netrc 
file parser for Go programming language
https://bugzilla.redhat.com/show_bug.cgi?id=1486510

   What|Removed |Added

 Status|ON_QA   |CLOSED
 Resolution|--- |ERRATA



-- 
You are receiving this mail because:
You are on the CC list for the bug.
You are always notified about changes to this product and component
___
package-review mailing list -- package-review@lists.fedoraproject.org
To unsubscribe send an email to package-review-le...@lists.fedoraproject.org

[Bug 1336168] Review Request: git-lfs - Git extension for versioning large files

[bugzilla]

https://bugzilla.redhat.com/show_bug.cgi?id=1336168
Bug 1336168 depends on bug 1486511, which changed state.

Bug 1486511 Summary: Review Request: golang-github-ThomsonReutersEikon-go-ntlm 
- Native implementation of NTLM for Go
https://bugzilla.redhat.com/show_bug.cgi?id=1486511

   What|Removed |Added

 Status|ON_QA   |CLOSED
 Resolution|--- |ERRATA



-- 
You are receiving this mail because:
You are on the CC list for the bug.
You are always notified about changes to this product and component
___
package-review mailing list -- package-review@lists.fedoraproject.org
To unsubscribe send an email to package-review-le...@lists.fedoraproject.org

[Bug 1336168] Review Request: git-lfs - Git extension for versioning large files

[bugzilla]

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

Elliott Sales de Andrade  changed:

   What|Removed |Added

  Alias|git-lfs |



-- 
You are receiving this mail because:
You are on the CC list for the bug.
You are always notified about changes to this product and component
___
package-review mailing list -- package-review@lists.fedoraproject.org
To unsubscribe send an email to package-review-le...@lists.fedoraproject.org

[Bug 1336168] Review Request: git-lfs - Git extension for versioning large files

[bugzilla]

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

Elliott Sales de Andrade  changed:

   What|Removed |Added

 Status|NEW |CLOSED
 Resolution|--- |DUPLICATE
Last Closed||2017-08-31 18:07:22



--- Comment #13 from Elliott Sales de Andrade  ---
Replaced by new RR.

*** This bug has been marked as a duplicate of bug 1487430 ***

-- 
You are receiving this mail because:
You are on the CC list for the bug.
You are always notified about changes to this product and component
___
package-review mailing list -- package-review@lists.fedoraproject.org
To unsubscribe send an email to package-review-le...@lists.fedoraproject.org

[Bug 1336168] Review Request: git-lfs - Git extension for versioning large files

[bugzilla]

https://bugzilla.redhat.com/show_bug.cgi?id=1336168
Bug 1336168 depends on bug 1336161, which changed state.

Bug 1336161 Summary: Review Request: golang-github-ThomsonReutersEikon-go-ntlm 
- Native implementation of NTLM for Go
https://bugzilla.redhat.com/show_bug.cgi?id=1336161

   What|Removed |Added

 Status|ASSIGNED|CLOSED
 Resolution|--- |DUPLICATE



-- 
You are receiving this mail because:
You are on the CC list for the bug.
You are always notified about changes to this product and component
___
package-review mailing list -- package-review@lists.fedoraproject.org
To unsubscribe send an email to package-review-le...@lists.fedoraproject.org

[Bug 1336168] Review Request: git-lfs - Git extension for versioning large files

[bugzilla]

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

Elliott Sales de Andrade  changed:

   What|Removed |Added

 Depends On||1486511




Referenced Bugs:

https://bugzilla.redhat.com/show_bug.cgi?id=1486511
[Bug 1486511] Review Request: golang-github-ThomsonReutersEikon-go-ntlm -
Native implementation of NTLM for Go
-- 
You are receiving this mail because:
You are on the CC list for the bug.
You are always notified about changes to this product and component
___
package-review mailing list -- package-review@lists.fedoraproject.org
To unsubscribe send an email to package-review-le...@lists.fedoraproject.org

[Bug 1336168] Review Request: git-lfs - Git extension for versioning large files

[bugzilla]

https://bugzilla.redhat.com/show_bug.cgi?id=1336168
Bug 1336168 depends on bug 1336164, which changed state.

Bug 1336164 Summary: Review Request: golang-github-bgentry-go-netrc - netrc 
file parser for Go programming language
https://bugzilla.redhat.com/show_bug.cgi?id=1336164

   What|Removed |Added

 Status|ASSIGNED|CLOSED
 Resolution|--- |DUPLICATE



-- 
You are receiving this mail because:
You are on the CC list for the bug.
You are always notified about changes to this product and component
___
package-review mailing list -- package-review@lists.fedoraproject.org
To unsubscribe send an email to package-review-le...@lists.fedoraproject.org

[Bug 1336168] Review Request: git-lfs - Git extension for versioning large files

[bugzilla]

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

Elliott Sales de Andrade  changed:

   What|Removed |Added

 Depends On||1486510




Referenced Bugs:

https://bugzilla.redhat.com/show_bug.cgi?id=1486510
[Bug 1486510] Review Request: golang-github-bgentry-go-netrc - netrc file
parser for Go programming language
-- 
You are receiving this mail because:
You are on the CC list for the bug.
You are always notified about changes to this product and component
___
package-review mailing list -- package-review@lists.fedoraproject.org
To unsubscribe send an email to package-review-le...@lists.fedoraproject.org

[Bug 1336168] Review Request: git-lfs - Git extension for versioning large files

[bugzilla]

https://bugzilla.redhat.com/show_bug.cgi?id=1336168
Bug 1336168 depends on bug 1336159, which changed state.

Bug 1336159 Summary: Review Request: golang-github-rubyist-tracerx - Output 
tracing information in your Go app based on environment variables
https://bugzilla.redhat.com/show_bug.cgi?id=1336159

   What|Removed |Added

 Status|POST|CLOSED
 Resolution|--- |RAWHIDE



-- 
You are receiving this mail because:
You are on the CC list for the bug.
You are always notified about changes to this product and component
___
package-review mailing list -- package-review@lists.fedoraproject.org
To unsubscribe send an email to package-review-le...@lists.fedoraproject.org

[Bug 1336168] Review Request: git-lfs - Git extension for versioning large files

[bugzilla]

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



--- Comment #12 from cl...@redhat.com ---
We will need to resolve blockers first:

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

and

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

-- 
You are receiving this mail because:
You are on the CC list for the bug.
You are always notified about changes to this product and component
___
package-review mailing list -- package-review@lists.fedoraproject.org
To unsubscribe send an email to package-review-le...@lists.fedoraproject.org

[Bug 1336168] Review Request: git-lfs - Git extension for versioning large files

[bugzilla]

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

cl...@redhat.com changed:

   What|Removed |Added

 CC||cl...@redhat.com
   Assignee|nob...@fedoraproject.org|cl...@redhat.com



-- 
You are receiving this mail because:
You are on the CC list for the bug.
You are always notified about changes to this product and component
___
package-review mailing list -- package-review@lists.fedoraproject.org
To unsubscribe send an email to package-review-le...@lists.fedoraproject.org

[Bug 1336168] Review Request: git-lfs - Git extension for versioning large files

[bugzilla]

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



--- Comment #11 from Elliott Sales de Andrade  ---
Here's an updated build spec:

Spec URL:
https://copr-be.cloud.fedoraproject.org/results/qulogic/git-lfs/fedora-26-x86_64/00584845-git-lfs/git-lfs.spec
SRPM URL:
https://copr-be.cloud.fedoraproject.org/results/qulogic/git-lfs/fedora-26-x86_64/00584845-git-lfs/git-lfs-2.2.1-1.fc26.src.rpm

-- 
You are receiving this mail because:
You are on the CC list for the bug.
You are always notified about changes to this product and component
___
package-review mailing list -- package-review@lists.fedoraproject.org
To unsubscribe send an email to package-review-le...@lists.fedoraproject.org

[Bug 1336168] Review Request: git-lfs - Git extension for versioning large files

[bugzilla]

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

Igor Gnatenko  changed:

   What|Removed |Added

  Flags|needinfo?(ignatenko@redhat. |
   |com)|



--- Comment #10 from Igor Gnatenko  ---
Sorry, but golang packaging just cumbersome and I'm not interested in being POC
for any of golang packages.

Sorry, guys. Still happy to co-maintain, but not primary maintainer.

Feel free to takeover this review request (note that you don't need to create
new one, but just give direct links for specs/srpms here and continue review).

-- 
You are receiving this mail because:
You are on the CC list for the bug.
You are always notified about changes to this product and component
___
package-review mailing list -- package-review@lists.fedoraproject.org
To unsubscribe send an email to package-review-le...@lists.fedoraproject.org

[Bug 1336168] Review Request: git-lfs - Git extension for versioning large files

[bugzilla]

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

Elliott Sales de Andrade  changed:

   What|Removed |Added

  Flags||needinfo?(ignatenko@redhat.
   ||com)



--- Comment #9 from Elliott Sales de Andrade  ---
2.0.2 spec with patch to build on Go 1.7 is here:
https://gitlab.com/qulogic-rpms/git-lfs/tree/master/git-lfs

Are you going to continue with this request, Igor?

-- 
You are receiving this mail because:
You are on the CC list for the bug.
You are always notified about changes to this product and component
___
package-review mailing list -- package-review@lists.fedoraproject.org
To unsubscribe send an email to package-review-le...@lists.fedoraproject.org

[Bug 1336168] Review Request: git-lfs - Git extension for versioning large files

[bugzilla]

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

Sascha Spreitzer  changed:

   What|Removed |Added

 Status|ASSIGNED|NEW
   Assignee|sspre...@redhat.com |nob...@fedoraproject.org
  Flags|needinfo?(sspreitz@redhat.c |
   |om) |
   |needinfo?(ignatenko@redhat. |
   |com)|



--- Comment #8 from Sascha Spreitzer  ---
Hi 

I am sorry i have to step down on this package review, due to lack of time.

Kind regards
Sascha

-- 
You are receiving this mail because:
You are on the CC list for the bug.
You are always notified about changes to this product and component
___
package-review mailing list -- package-review@lists.fedoraproject.org
To unsubscribe send an email to package-review-le...@lists.fedoraproject.org

[Bug 1336168] Review Request: git-lfs - Git extension for versioning large files

[bugzilla]

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



--- Comment #7 from Elliott Sales de Andrade  ---
git-lfs v2.0.1 is available in my copr I linked above. 2.0.2 currently fails to
build on F25 as it appears to use a function from Go 1.8.

-- 
You are receiving this mail because:
You are on the CC list for the bug.
You are always notified about changes to this product and component
___
package-review mailing list -- package-review@lists.fedoraproject.org
To unsubscribe send an email to package-review-le...@lists.fedoraproject.org

[Bug 1336168] Review Request: git-lfs - Git extension for versioning large files

[bugzilla]

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

Elliott Sales de Andrade  changed:

   What|Removed |Added

 Depends On|1336166 |
   |(golang-github-technoweenie |
   |-assert)|




Referenced Bugs:

https://bugzilla.redhat.com/show_bug.cgi?id=1336166
[Bug 1336166] Review Request: golang-github-technoweenie-assert - Asserts
to Go testomg
-- 
You are receiving this mail because:
You are on the CC list for the bug.
You are always notified about changes to this product and component
___
package-review mailing list -- package-review@lists.fedoraproject.org
To unsubscribe send an email to package-review-le...@lists.fedoraproject.org

[Bug 1336168] Review Request: git-lfs - Git extension for versioning large files

[bugzilla]

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

Warren Togami  changed:

   What|Removed |Added

 CC||wtog...@gmail.com



--- Comment #6 from Warren Togami  ---
Any update on this?  git-lfs is v2.0.2 upstream.

-- 
You are receiving this mail because:
You are on the CC list for the bug.
You are always notified about changes to this product and component
___
package-review mailing list -- package-review@lists.fedoraproject.org
To unsubscribe send an email to package-review-le...@lists.fedoraproject.org

[Bug 1336168] Review Request: git-lfs - Git extension for versioning large files

[bugzilla]

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

James Hogarth  changed:

   What|Removed |Added

 CC||james.hoga...@gmail.com
  Flags||needinfo?(ignatenko@redhat.
   ||com)



--- Comment #5 from James Hogarth  ---
Igor are you still planning on packaging this?

-- 
You are receiving this mail because:
You are on the CC list for the bug.
You are always notified about changes to this product and component
___
package-review mailing list -- package-review@lists.fedoraproject.org
To unsubscribe send an email to package-review-le...@lists.fedoraproject.org

[Bug 1336168] Review Request: git-lfs - Git extension for versioning large files

[bugzilla]

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

Elliott Sales de Andrade  changed:

   What|Removed |Added

 CC||quantum.anal...@gmail.com



--- Comment #4 from Elliott Sales de Andrade  ---
I've tried to update this to 1.5.5 [1,2]. Bug 1336166 can be dropped because
it's no longer a dependency. It will newly require
golang-github-xeipuuv-gojsonschema [3], golang-github-xeipuuv-gojsonpointer
[4], and golang-github-xeipuuv-gojsonreference [5].

Since those last three are still in testing, my copr build will probably fail,
but the queue is a bit weird today.

[1]
https://gitlab.com/QuLogic/rpmbuild-specs/commit/9ca0fd8b1ed92c993e053641dfdcc56d38f8f755
[2] https://copr.fedorainfracloud.org/coprs/qulogic/git-lfs/build/511065/
[3] https://bugzilla.redhat.com/show_bug.cgi?id=1405074 
[4] https://bugzilla.redhat.com/show_bug.cgi?id=1405067
[5] https://bugzilla.redhat.com/show_bug.cgi?id=1405072

-- 
You are receiving this mail because:
You are on the CC list for the bug.
You are always notified about changes to this product and component
___
package-review mailing list -- package-review@lists.fedoraproject.org
To unsubscribe send an email to package-review-le...@lists.fedoraproject.org

[Bug 1336168] Review Request: git-lfs - Git extension for versioning large files

[bugzilla]

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

Jan Chaloupka  changed:

   What|Removed |Added

 CC||jchal...@redhat.com



--- Comment #3 from Jan Chaloupka  ---
Summary:
- please provide %{commit}, %{shortcommit}, %{provider} and the usual macros
for Go packages. They are handful (and needed) for tooling running over Go
projects packaged in Fedora.
- as long as the package is for Fedora only, BuildRequires: 
compiler(go-compiler) is fine. It will not work for epel6, epel7 and RHEL7 as
compiler(go-compiler) is not defined there.
- provide devel and unit-test subpackages as well (for analysis carried over Go
projects in distribution)
- to build from de-bundled dependencies, all imports prefixed with
"github.com/github/git-lfs/vendor/_nuts/" must be updated. (Remove the prefix).
U can use: sed -i "s/github\.com\/github\/git-lfs\/vendor\/_nuts\///" $(find .
-name '*.go')

After removing the vendor prefix, the following list of deps remains:

BuildRequires: golang(github.com/ThomsonReutersEikon/go-ntlm/ntlm)
BuildRequires: golang(github.com/bgentry/go-netrc/netrc)
BuildRequires: golang(github.com/cheggaaa/pb)
BuildRequires: golang(github.com/kr/pty)
BuildRequires: golang(github.com/olekukonko/ts)
BuildRequires: golang(github.com/rubyist/tracerx)
BuildRequires: golang(github.com/spf13/cobra)

-- 
You are receiving this mail because:
You are on the CC list for the bug.
You are always notified about changes to this product and component
___
package-review mailing list
package-review@lists.fedoraproject.org
https://lists.fedoraproject.org/admin/lists/package-review@lists.fedoraproject.org

[Bug 1336168] Review Request: git-lfs - Git extension for versioning large files

[bugzilla]

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

Igor Gnatenko  changed:

   What|Removed |Added

  Flags||needinfo?(sspreitz@redhat.c
   ||om)



--- Comment #2 from Igor Gnatenko  ---
(In reply to Sascha Spreitzer from comment #1)
> Hi Igor
> 
> I will be reviewing your request.
It's good, what about other packages? ;)
> 
> Kind regards
> Sascha

-- 
You are receiving this mail because:
You are on the CC list for the bug.
You are always notified about changes to this product and component
___
package-review mailing list
package-review@lists.fedoraproject.org
https://lists.fedoraproject.org/admin/lists/package-review@lists.fedoraproject.org

[Bug 1336168] Review Request: git-lfs - Git extension for versioning large files

[bugzilla]

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

Sascha Spreitzer  changed:

   What|Removed |Added

 Status|NEW |ASSIGNED
 CC||sspre...@redhat.com
   Assignee|nob...@fedoraproject.org|sspre...@redhat.com
  Flags||fedora-review?



--- Comment #1 from Sascha Spreitzer  ---
Hi Igor

I will be reviewing your request.

Kind regards
Sascha

-- 
You are receiving this mail because:
You are on the CC list for the bug.
You are always notified about changes to this product and component
___
package-review mailing list
package-review@lists.fedoraproject.org
https://lists.fedoraproject.org/admin/lists/package-review@lists.fedoraproject.org

[Bug 1336168] Review Request: git-lfs - Git extension for versioning large files

[bugzilla]

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

Igor Gnatenko  changed:

   What|Removed |Added

  Alias||git-lfs



-- 
You are receiving this mail because:
You are on the CC list for the bug.
You are always notified about changes to this product and component
___
package-review mailing list
package-review@lists.fedoraproject.org
http://lists.fedoraproject.org/admin/lists/package-review@lists.fedoraproject.org

[Bug 1336168] Review Request: git-lfs - Git extension for versioning large files

[bugzilla]

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

Igor Gnatenko  changed:

   What|Removed |Added

 Depends On||1336159
   ||(golang-github-rubyist-trac
   ||erx), 1336161, 1336164
   ||(golang-github-bgentry-go-n
   ||etrc), 1336166
   ||(golang-github-technoweenie
   ||-assert)




Referenced Bugs:

https://bugzilla.redhat.com/show_bug.cgi?id=1336159
[Bug 1336159] Review Request: golang-github-rubyist-tracerx - Output
tracing information in your Go app based on environment variables
https://bugzilla.redhat.com/show_bug.cgi?id=1336161
[Bug 1336161] Review Request: golang-github-ThomsonReutersEikon-go-ntlm -
Native implementation of NTLM for Go
https://bugzilla.redhat.com/show_bug.cgi?id=1336164
[Bug 1336164] Review Request: golang-github-bgentry-go-netrc - netrc file
parser for Go programming language
https://bugzilla.redhat.com/show_bug.cgi?id=1336166
[Bug 1336166] Review Request: golang-github-technoweenie-assert - Asserts
to Go testomg
-- 
You are receiving this mail because:
You are on the CC list for the bug.
You are always notified about changes to this product and component
___
package-review mailing list
package-review@lists.fedoraproject.org
http://lists.fedoraproject.org/admin/lists/package-review@lists.fedoraproject.org