Red Hat Bugzilla – Bug 212513
Review Request: sparse - source code semantec parser used by the Linux kernel
Last modified: 2007-11-30 17:11:46 EST
Spec URL: http://domsch.com/linux/fedora/extras/sparse/sparse.spec
SRPM URL: http://domsch.com/linux/fedora/extras/sparse/sparse-0-0.1.20061026git.src.rpm
Sparse is a semantic parser of source files: it's neither a compiler
(although it could be used as a front-end for one) nor is it a
preprocessor (although it contains as a part of it a preprocessing
It is meant to be a small - and simple - library. Scanty and meager,
and partly because of that easy to use. It has one mission in life:
create a semantic parse tree for some arbitrary user for further
analysis. It's not a tokenizer, nor is it some generic context-free
parser. In fact, context (semantics) is what it's all about - figuring
out not just what the grouping of tokens are, but what the _types_ are
that the grouping implies.
Sparse is primarily used in the development and debugging of the Linux kernel.
*** Bug 185325 has been marked as a duplicate of this bug. ***
bkyoung, I'm sorry I didn't see your review request when I went looking for
this. I believe my spec is cleaner and per packaging guidelines, which should
accelerate the process.
Is there a reason not to use the snapshot tarballs that Dave Jones provides for
sparse at http://www.codemonkey.org.uk/projects/git-snapshots/sparse/ ? Since
sparse really doesn't do releases, that is about as close as it comes. Also,
does "git" really have to be in the alphatag?
Other than that, the package looks fairly clean. rpmlint doesn't complain about
anything, license is fine, and the package builds on i386 just fine. I'll test
re: tarballs - doesn't honestly matter to me if the package has a daily snapshot
tarball or a git clone'd tarball from a given day. I can put a comment in that
snapshot tarballs are also available at DaveJ's site, or can use his tarballs
directly, either way.
re: git in the alphatag, that's a figment of git clone method above and
Packaging/NamingGuidelines says to use it. I suppose if I used one of the
snapshot tarballs, that could be dropped, though NamingGuidelines doesn't really
cover this scenario exactly (a non-project-released snapshot tarball). Spot?
Thanks for the additional review and comments. Package builds and works on
x86_64 for me too btw; I haven't tried ppc.
If you're using a manually created git clone, then yeah, the git in the alphatag
should be there. If you use one of the snapshots from upstream, then you don't
(In reply to comment #4)
> re: tarballs - doesn't honestly matter to me if the package has a daily snapshot
> tarball or a git clone'd tarball from a given day. I can put a comment in that
> snapshot tarballs are also available at DaveJ's site, or can use his tarballs
> directly, either way.
> re: git in the alphatag, that's a figment of git clone method above and
> Packaging/NamingGuidelines says to use it. I suppose if I used one of the
> snapshot tarballs, that could be dropped, though NamingGuidelines doesn't really
> cover this scenario exactly (a non-project-released snapshot tarball). Spot?
Well, the only reason I suggested the DaveJ snapshots was to basically avoid the
git in the alphatag. It's not really a big deal to me.
> Thanks for the additional review and comments. Package builds and works on
> x86_64 for me too btw; I haven't tried ppc.
I don't expect problems on ppc. It should just work.
* Package and spec named appropriately: See note below
* Spec file is legible and in Am. English
* Source matches upstream: See note below
* No unnecessary BuildRequires
* No locales
* No shared libraries in the default linker path
* RPM_BUILD_ROOT cleaned where appropriate
* Not relocatable
* No duplicate %files
* File permissions look ok
* No need for a -devel subpackage
* Not a gui program; no need for a .desktop file
* Consistent use of macros
* Does not own any directories that it should not own.
Note: I've sent an email to the upstream developers discussing a possible
official release. If that comes to pass, we can use those for this package.
Until then, git or tarball snapshots will work just fine.
This package builds fine on ppc. This passes review.
packages built and released, closing.
Please transfer ownership of sparse on all branches to Roland McGrath.