Bug 225695
Summary: | Merge Review: diffstat | ||
---|---|---|---|
Product: | [Fedora] Fedora | Reporter: | Nobody's working on this, feel free to take it <nobody> |
Component: | Package Review | Assignee: | Rafał Psota <rafalzaq> |
Status: | CLOSED RAWHIDE | QA Contact: | Fedora Package Reviews List <fedora-package-review> |
Severity: | medium | Docs Contact: | |
Priority: | medium | ||
Version: | rawhide | CC: | twaugh |
Target Milestone: | --- | Flags: | rafalzaq:
fedora-review+
|
Target Release: | --- | ||
Hardware: | All | ||
OS: | Linux | ||
Whiteboard: | |||
Fixed In Version: | Doc Type: | Bug Fix | |
Doc Text: | Story Points: | --- | |
Clone Of: | Environment: | ||
Last Closed: | 2007-03-29 09:06:29 UTC | Type: | --- |
Regression: | --- | Mount Type: | --- |
Documentation: | --- | CRM: | |
Verified Versions: | Category: | --- | |
oVirt Team: | --- | RHEL 7.3 requirements from Atomic Host: | |
Cloudforms Team: | --- | Target Upstream Version: | |
Embargoed: |
Description
Nobody's working on this, feel free to take it
2007-01-31 18:27:19 UTC
Needs work: * Source 0 is not available (ftp://invisible-island.net/diffstat/diffstat-1.43.tar.gz) * BuildRoot should be %{_tmppath}/%{name}-%{version}-%{release}-root-%(%{__id_u} -n) * BuildRequires: gzip and bzip2 should not be included (exceptions) * Missing SMP flags. If it doesn't build with it, please add a comment * The %makeinstall macro should not be used * deffattr should be (-,root,root,-) instead of (-,root,root) * rpmlint output: Source RPM: W: diffstat summary-ended-with-dot A utility which provides statistics based on the output of diff. W: diffstat invalid-license distributable W: diffstat macro-in-%changelog configure rpmlint of diffstat: W: diffstat summary-ended-with-dot A utility which provides statistics based on the output of diff. W: diffstat invalid-license distributable rpmlint of diffstat-debuginfo: W: diffstat-debuginfo invalid-license distributable Is there a text of license somewhere? > * Source 0 is not available (ftp://invisible-island.net/diffstat/diffstat-1.43.tar.gz) The upstream source is named 'diffstat.tar.gz'. How should I change Source0? > W: diffstat-debuginfo invalid-license distributable > > Is there a text of license somewhere? No, there is not one that I know of. The web page says "All are copyrighted, freely distributable works". All other changes applied. Tagged and built as 1.43-2.fc7. (In reply to comment #2) > > W: diffstat-debuginfo invalid-license distributable > > > > Is there a text of license somewhere? > > No, there is not one that I know of. The web page says "All are copyrighted, > freely distributable works". That doesn't look to be OSI-compatible, since it seems not to be freely modifiable. Okay, I've tracked down this from diffstat.c: /****************************************************************************** * Copyright 1994-2005,2006 by Thomas E. Dickey * * All Rights Reserved. * * * * Permission to use, copy, modify, and distribute this software and its * * documentation for any purpose and without fee is hereby granted, provided * * that the above copyright notice appear in all copies and that both that * * copyright notice and this permission notice appear in supporting * * documentation, and that the name of the above listed copyright holder(s) * * not be used in advertising or publicity pertaining to distribution of the * * software without specific, written prior permission. * * * * THE ABOVE LISTED COPYRIGHT HOLDER(S) DISCLAIM ALL WARRANTIES WITH REGARD * * TO THIS SOFTWARE, INCLUDING ALL IMPLIED WARRANTIES OF MERCHANTABILITY AND * * FITNESS, IN NO EVENT SHALL THE ABOVE LISTED COPYRIGHT HOLDER(S) BE LIABLE * * FOR ANY SPECIAL, INDIRECT OR CONSEQUENTIAL DAMAGES OR ANY DAMAGES * * WHATSOEVER RESULTING FROM LOSS OF USE, DATA OR PROFITS, WHETHER IN AN * * ACTION OF CONTRACT, NEGLIGENCE OR OTHER TORTIOUS ACTION, ARISING OUT OF OR * * IN CONNECTION WITH THE USE OR PERFORMANCE OF THIS SOFTWARE. * ******************************************************************************/ That is BSD-like or MIT-like (or even BSD or MIT), I don't remember exactly, and clearly right. (In reply to comment #2) > The upstream source is named 'diffstat.tar.gz'. How should I change Source0? diffstat.tar.gz is only a symlink to diffstat-1.43.tgz. Just change tar.gz to tgz. I think you should copy the license text to LICENSE (COPYING, etc.) file and include it as another Source (that file goes to %doc of course). Oh yes, so it is. Tagged and built as 1.43-2.fc7. rpmlint is not silent: W: diffstat invalid-license distributable #should be Distributable In specfile: URL: http://invisible-island.net/diffstat # Note that the upstream tarball is unversioned, just %{name}.tar.gz Source0: ftp://invisible-island.net/diffstat/%{name}-%{version}.tgz IMO this comment is no longer needed. Thanks. Tagged and built as 1.43-4.fc7. In description you have something like this: "You'll need to also install diffutils.". I think it should be listed in Requires. I've removed that sentence. I haven't added a 'Requires' tag for diffutils because there is no dependency there -- it is entirely possible that you might want to run diffstat on a patch you've received by email without having diffutils installed, and that works just fine. Tagged and built as 1.43-5.fc7. Everything looks good so this package is APPROVED. |