[Bug 1432214] Review Request: reg - Docker registry v2 command line client.

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



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

Adam Miller <admiller@xxxxxxxxxx> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
              Flags|needinfo?(admiller@redhat.c |
                   |om)                         |



--- Comment #2 from Adam Miller <admiller@xxxxxxxxxx> ---
1) The app cannot, it's authored upstream expecting to run in a container and
it thinks that it has free reign over / and I actually moved it to /var/
because that seemed like a better idea. I do agree it's kind of weird. It's
possible this violates the FHS and if so, then I'll work on patching the
upstream code to cater to a better directory layout.

2) Would HTML/CSS content make sense in /var/lib/ ?

3) I don't believe we can get rid of bundled libs, this duplicates golang
packages already in Fedora but needs different versions of them.

----

4) Yes, I should do that instead of how I did. Will fix.

5) Not a typo, I followed the RHEL docs ->
https://access.redhat.com/documentation/en-US/Red_Hat_Enterprise_Linux/7/html/System_Administrators_Guide/sect-Managing_Services_with_systemd-Unit_Files.html

6) +1 it should be global

7) Will remove the .

8) I didn't get that lint warning, will check again to make sure I'm not using
tabs v spaces.


----

9) Yes, it's a mess and it doesn't have a good debugger. I have trouble taking
it seriously.

-- 
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@xxxxxxxxxxxxxxxxxxxxxxx
To unsubscribe send an email to package-review-leave@xxxxxxxxxxxxxxxxxxxxxxx




[Index of Archives]     [Fedora Legacy]     [Fedora Desktop]     [Fedora SELinux]     [Yosemite News]     [KDE Users]     [Fedora Tools]