Project

General

Profile

Feature #5470

libz should be part of illumos

Added by Gary Mills over 5 years ago. Updated over 5 years ago.

Status:
Feedback
Priority:
Normal
Assignee:
Category:
lib - userland libraries
Start date:
2014-12-22
Due date:
% Done:

90%

Estimated time:
Difficulty:
Medium
Tags:
needs-triage
Gerrit CR:

Description

The problem here is that many illumos commands and libraries link with libz, but libz only exists on the build host. This means that libz requires libc on the build host but that version of libc may be different from the one in illumos. Makefiles at these locations link with libz:


usr/src/cmd/boot/fiocompress
usr/src/tools/ctf/dump
usr/src/cmd/ssh/ssh
usr/src/ssh/ssh-keyscan
usr/src/ssh/ssh-keysign
usr/src/cmd/boot/bootadm
usr/src/tools/ctf/cvt
usr/src/cmd/ssh/libssh
usr/src/cmd/ssh/sshd
usr/src/lib/libzpool

It should be easy to incorporate libz, as it's a separate product that only requires libc. The alternative of removing all of the components that link with libz is much more difficult.


Related issues

Is duplicate of illumos gate - Bug #1002: Integrate zlibNew2011-05-08

Actions

History

#1

Updated by Andrew Stormont over 5 years ago

Agreed. It should be part of illumos.

#2

Updated by Gary Mills over 5 years ago

  • Status changed from New to Feedback
  • Assignee set to Gary Mills
  • % Done changed from 0 to 10

This bug report is partially a duplicate of bug #1002. It's unfortunate that this library has two names.

#3

Updated by Gary Mills over 5 years ago

  • % Done changed from 10 to 90

I've had successful builds on x86 and SPARC.

#4

Updated by Gordon Ross over 5 years ago

While I appreciate the efforts to deal with these dependencies, before we start copying more code into illumos-gate, I'd like a decision on how we handle these things. See my email to developers today: Proposing a new approach for things like OpenSSL

#5

Updated by Gary Mills over 5 years ago

Since most people are satisfied with the status quo, I'm going to abandon this project. It seems that illumos requiring zlib from the build host is not a problem after all.

Could somebody please close this bug report. I can't do it myself.

#6

Updated by Andrew Stormont over 5 years ago

I don't think you should give up on these changes yet. I am definitely not happy with the status quo.

Also available in: Atom PDF