[Bug 230275] Review Request: varnish - High-performance HTTP accelerator

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

 



Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug report.

Summary: Review Request: varnish - High-performance HTTP accelerator


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





------- Additional Comments From ingvar@xxxxxxxxx  2007-04-18 02:23 EST -------
* Kevin Fenzi
> 6. ok on static libs, but did you really want to disable the dynamic
> libs as well? Does anything known link against the varnish dynamic libs?
> Is there a reason to not ship them?

The dynamic libraries are there allright. I just removed the ".so"
symlinks that points to them. These symlinks are usually part of the
-devel package in redhat-based systems, it I have understood
correctly. Varnish itself links to the main-versioned symlink.

$ ls -l /usr/lib64/*varnish*so
lrwxrwxrwx  1 root root 22 feb 28 14:17 /usr/lib64/libvarnishapi.so ->
libvarnishapi.so.0.0.0
lrwxrwxrwx  1 root root 19 feb 28 14:17 /usr/lib64/libvarnish.so ->
libvarnish.so.0.0.0

$ ldd /usr/sbin/varnishd  | grep usr | cut -d '(' -f 1
        libvarnish.so.0 => /usr/lib64/libvarnish.so.0
        libvcl.so.0 => /usr/lib64/libvcl.so.0

So for varnish itself, the .so symlinks are not necessary, though they
might be valuable if anybody want to develop things on top of varnish
later. Thus, they might belong in a future -devel package.

> 7. ok. looks good. You shouldn't need the LICENSE in all the
> subpackages, but it's not a blocker if you want to do so. The
> rpmlint warning about no docs can be ignored for devel subpackages.

I get this warm fuzzy feeling when I get no errors or warnings, you
see. Also note that "all the subpackages" at the moment counts one :-)

> If you really want to ship the INSTALL thats fine I would think.

Good

> So the only final question I see here is if the devel package should
> be shipped or not. 

Well, I don't know of anyone using varnish technology for anything but
varnish, so for an initial release, I'd say it's not necessary. A
devel package should perhaps include some header files and some
hacking starting point docs too. If we can push it for later, I'll ask
some of the developers for this.

Ingvar


-- 
Configure bugmail: https://bugzilla.redhat.com/bugzilla/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
You are the QA contact for the bug, or are watching the QA contact.

_______________________________________________
Fedora-package-review mailing list
Fedora-package-review@xxxxxxxxxx
http://www.redhat.com/mailman/listinfo/fedora-package-review

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