Page MenuHomeSolus

build and add minizip from contrib
ClosedPublic

Authored by curantes on Aug 7 2017, 8:20 PM.
Referenced Files
F11005714: D783.diff
Tue, Aug 1, 9:34 AM
F10897503: D783.id1714.diff
Jul 2 2023, 7:23 AM
F10876018: D783.id1714.diff
Jun 20 2023, 8:43 AM
F10865466: D783.diff
Jun 17 2023, 11:42 AM
F10832182: D783.id1714.diff
Jun 5 2023, 10:05 PM
F10829377: D783.id3081.diff
Jun 4 2023, 11:58 AM
F10828533: D783.id2741.diff
Jun 4 2023, 9:40 AM
F10828022: D783.id1714.diff
Jun 4 2023, 8:12 AM
Subscribers

Details

Summary

Build and add minizip from contrib depends on T2545

Signed-off-by: Matthias Eliasson <matthias.eliasson@gmail.com>

Test Plan

I have tested to install and the libs are located the same place. Also made sure to split minizip to an own package (pattern).
No further tests have beem made.

Diff Detail

Repository
R3325 zlib
Lint
Lint Not Applicable
Unit
Tests Not Applicable

Event Timeline

Not sure that I like the fact the subpackages aren't split off into further devels

In D783#11250, @ikey wrote:

Not sure that I like the fact the sub-packages aren't split off into further devels

I was actually a bit uncertain how to split this up. One way was to create a whole separate package repository for minizip but since this have been moved to zlib repository upstream for some time I also thought we could go with patterns and include this here. Also since minizip is just a lib we might just rename the pattern above and suffix the -devel and skip minizip?

Otherwise I am not sure how we would further split it.

@ikey: Do you have any suggestion in how you want minizip packaged?

Interested in getting this in for Chromium

Check the contents of zlib and zlib-devel if you still aren't sure

package.yml
20–23

It would be something like this:

- minizip:
    - /usr/lib64/libminizip.so.*
- minizip-devel:
    - /usr/include/minizip/*
    - /usr/lib64/*.a
    - /usr/lib64/libminizip.so
    - /usr/lib64/pkgconfig/minizip.pc

Added chromium as a child dep as it bundles it's own zlib with minizip support, but ideally I would like the use the system library.

split minizip to devel pattern as well according to pointers from @ikey

This revision is now accepted and ready to land.Oct 27 2017, 9:55 PM
This revision was automatically updated to reflect the committed changes.