[Bug 479413] Review Request: pywebdav - WebDAV library for Python

[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.


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





--- Comment #2 from Dan Horák <dan@xxxxxxxx>  2009-01-10 10:10:57 EDT ---
(In reply to comment #1)
> MUST: Each package must consistently use macros.
> 
> Not really. For example, consider the line
> %{__python} setup.py install -O1 --skip-build --root $RPM_BUILD_ROOT
> Which should be:
> %{__python} setup.py install -O1 --skip-build --root %{buildroot}
> 
> even though the first is given as an example in the documentation.

Using the first variant is completely legal. The consistency here means that
$RPM_BUILD_ROOT and %{buildroot} should not be mixed inside one spec file.

> Summary - MINOR issues with use of macros in spec file.
> 
> I am curious though as to why the example server isn't included in either the
> same package, or in a documentation package. I would expect to see it in
> /usr/share/pywebdav-0.8. I really think this would be a worthwhile addition.

There are few reasons why not include the server by default:
- the server is not installed by the upstream setup utility
- I have a package that will use only the library
- distributing a network server has a broader consequences (security, etc.)
So until there is a real demand for this server I don't have an intent to
package it.

-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
You are on the CC list for the bug.

_______________________________________________
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]