[Bug 2159049] Review Request: UEFITool - UEFI firmware image viewer and editor

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

 



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

Petr Menšík <pemensik@xxxxxxxxxx> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
                 CC|                            |pemensik@xxxxxxxxxx
           Doc Type|---                         |If docs needed, set a value



--- Comment #2 from Petr Menšík <pemensik@xxxxxxxxxx> ---
The package bundles quite a lot of projects we have already have packaged on
Fedora.

- qhexedit2
- zlib
- (? edk2)
- lzma-sdk
- libtomcrypt

Has the issue been raised to upstream we would like to use external compiled
libraries instead of bundled ones? Is it confirmed all of bundled libraries has
local modifications required by the UEFITool package and cannot use existing
code?

I think it should be recommended to keep bundled code without modifications and
modify the usage in the project-only code. I think it would make harder and
harder to maintain those libraries current and adds more maintenance work in
the future. A link to upstream issue requesting using shared code instead of
static modified libraries would be nice.

Otherwise the package spec is well done.


-- 
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
https://bugzilla.redhat.com/show_bug.cgi?id=2159049
_______________________________________________
package-review mailing list -- package-review@xxxxxxxxxxxxxxxxxxxxxxx
To unsubscribe send an email to package-review-leave@xxxxxxxxxxxxxxxxxxxxxxx
Fedora Code of Conduct: https://docs.fedoraproject.org/en-US/project/code-of-conduct/
List Guidelines: https://fedoraproject.org/wiki/Mailing_list_guidelines
List Archives: https://lists.fedoraproject.org/archives/list/package-review@xxxxxxxxxxxxxxxxxxxxxxx
Do not reply to spam, report it: https://pagure.io/fedora-infrastructure/new_issue




[Index of Archives]     [Fedora Users]     [Fedora Desktop]     [Fedora SELinux]     [Yosemite Conditions]     [KDE Users]

  Powered by Linux