Note: This is a public test instance of Red Hat Bugzilla. The data contained within is a snapshot of the live data so any changes you make will not be reflected in the production Bugzilla. Email is disabled so feel free to test any aspect of the site that you want. File any problems you find or give feedback at bugzilla.redhat.com.

Bug 175237

Summary: Review Request: bzr - bazaar-ng distributed revision control system
Product: [Fedora] Fedora Reporter: Shahms E. King <shahms>
Component: Package ReviewAssignee: Jeffrey C. Ollie <jeff>
Status: CLOSED NEXTRELEASE QA Contact: David Lawrence <dkl>
Severity: medium Docs Contact:
Priority: medium    
Version: rawhideCC: fedora-extras-list, mkanat, mlh
Target Milestone: ---   
Target Release: ---   
Hardware: All   
OS: Linux   
Whiteboard:
Fixed In Version: Doc Type: Bug Fix
Doc Text:
Story Points: ---
Clone Of: Environment:
Last Closed: 2006-02-13 15:45:11 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:
Bug Depends On:    
Bug Blocks: 163779    

Description Shahms E. King 2005-12-07 22:44:58 UTC
Spec Url: http://shahms.mesd.k12.or.us/yum/packages/bzr.spec 
SRPM URL: http://shahms.mesd.k12.or.us/yum/packages/bzr-0.6.2-1.src.rpm
Description: Bazaar-NG, the next-generation distributed revision control system is where the (former) developers of the bazaar fork of GNU Arch are now focusing there efforts.  It is a distributed, friendly, simple and flexible revision control system written in Python.

Comment 1 Jeffrey C. Ollie 2006-01-14 15:20:57 UTC
Good:

      - MUST: rpmlint output:

E: bzr non-executable-script
/usr/lib/python2.4/site-packages/bzrlib/store/weave.py 0644
E: bzr non-executable-script /usr/lib/python2.4/site-packages/bzrlib/revfile.py 0644
E: bzr non-executable-script
/usr/lib/python2.4/site-packages/bzrlib/selftest/test_weave.py 0644
E: bzr non-executable-script /usr/lib/python2.4/site-packages/bzrlib/xml4.py 0644
E: bzr non-executable-script /usr/lib/python2.4/site-packages/bzrlib/xml5.py 0644
E: bzr non-executable-script /usr/lib/python2.4/site-packages/bzrlib/upgrade.py 0644
E: bzr non-executable-script /usr/lib/python2.4/site-packages/bzrlib/xml.py
0644E: bzr non-executable-script
/usr/lib/python2.4/site-packages/bzrlib/weave.py 0644

which I think can be ignored

      - MUST: The package is named according to the PackageNamingGuidelines.
      - MUST: The spec file name matches the base package %{name}, in the format
%{name}.spec
      - MUST: The package meets the PackagingGuidelines.
      - MUST: Bzr has a GPL license.
      - MUST: The License field in the package spec file matches the actual license.
      - MUST: The source package does not have a copy of the license, so it
isn't in %doc
      - MUST: The spec file is written in American English.
      - MUST: The spec file for the package is legible.
      - MUST: The sources used to build the package match the upstream source,
as provided in the spec URL.
      - MUST: The package successfully compiles and builds on i386/devel.
      - MUST: A package must not contain any BuildRequires that are listed in
the exceptions section of PackagingGuidelines.
      - MUST: All other Build dependencies must be listed in BuildRequires.
      - MUST: The spec file MUST handle locales properly. This is done by using
the %find_lang macro. Using %{_datadir}/locale/* is strictly forbidden.
      - MUST: The package does not contain shared libraries.
      - MUST: The package is not designed to be relocatable.
      - MUST: The package owns all of the directories that it cretes.
      - MUST: Package does not contain any duplicate files in the %files listing.
      - MUST: Permissions on files are set properly. %files section includes a
%defattr(...) line.
      - MUST: Package has a %clean section, which contains rm -rf $RPM_BUILD_ROOT.
      - MUST: Each package must consistently use macros, as described in the
macros section of PackagingGuidelines.
      - MUST: The package contains code.
      - MUST: Large documentation files should go in a -docs subpackage. (The
definition of large is left up to the packager's best judgement, but is not
restricted to size. Large can refer to either size or quantity)
      - MUST: If a package includes something as %doc, it must not affect the
runtime of the application. To summarize: If it is in %doc, the program must run
properly if it is not present.
      - MUST: Header files or static libraries must be in a -devel package.
      - MUST: Files used by pkgconfig (.pc files) must be in a -devel package.
      - MUST: If a package contains library files with a suffix (e.g.
libfoo.so.1.1), then library files that end in .so (without suffix) must go in a
-devel package.
      - MUST: In the vast majority of cases, devel packages must require the
base package using a fully versioned dependency.
      - MUST: Packages does not contain any .la libtool archives, these should
be removed in the spec.
      - MUST: Package does not include a GUI app.
      - MUST: Package does not own files or directories already owned by other
packages.
      - SHOULD: Builds in mock on devel/i386, devel/x86_64, FC4/x86_64
      - SHOULD: No scriptlets are used.
      - SHOULD: There are no subpackages.

Would be nice:

      - SHOULD: If the source package does not include license text(s) as a
separate file from upstream, the packager SHOULD query upstream to include it.
      - SHOULD: The description and summary sections in the package spec file
should contain translations for supported Non-English languages, if available.

Bad:

      - SHOULD: The reviewer should test that the package functions as
described. A package should not segfault instead of running, for example.

Every command results in the following error:

[jeff@max1 ~]$ bzr help init
bzr: ERROR: No module named configobj.configobj
  command: '/usr/bin/bzr' 'help' 'init'
      pwd: u'/home/jeff'
    error: exceptions.ImportError
  at /usr/lib/python2.4/site-packages/bzrlib/config.py line 62, in ?()
  see ~/.bzr.log for debug information

NEEDSWORK

Comment 2 Shahms E. King 2006-01-26 19:20:57 UTC
The errors are the result of an issue with the system library patch and the fact
that I had configobj installed locally.  Ideally, I'd package python-configobj
and just depend on that; but for now, I just use the one upstream bundles.
I can add a copy of the GPL to the package if it's really necessary, but cannot
do much about a non-English summary or description.

There will be updated files fixing the configobj problem shortly:
Spec Url: http://shahms.mesd.k12.or.us/yum/packages/bzr.spec 
SRPM URL: http://shahms.mesd.k12.or.us/yum/packages/bzr-0.6.2-2.src.rpm

Thanks for looking at the package.

Comment 3 Jeffrey C. Ollie 2006-01-27 15:09:44 UTC
The -2 package builds and works for me now (at least for a quick test), thanks!
 Getting a copy of the GPL into the tarball was just a suggestion to bring to
the upstream developers, as was the non-English description.

APPROVED


Comment 4 Jeffrey C. Ollie 2006-02-08 20:54:53 UTC
Shahms, I noticed that version 0.7 is out, are you going to update/get this into FE?

Comment 5 Shahms E. King 2006-02-08 21:55:16 UTC
Yeah, getting bzr into FE is high on my TODO list, but I've been swamped with
work and school, leaving little time for much else.  Barring unforeseen events,
I should have it updated and into FE by this weekend.

Comment 6 Matthew Hannigan 2006-05-10 23:43:04 UTC
bzr 0.8 is out and much improved.
it also likely to be very stable and used for a long time as it
is the version most likely to be included with ubuntu's 'long
term support' linux release.

Is 0.8 expected to make it into extras soon?


Comment 7 Matthew Hannigan 2006-05-10 23:44:12 UTC
bzr 0.8 is out and much improved.
it also likely to be very stable and used for a long time as it
is the version most likely to be included with ubuntu's 'long
term support' linux release.

Is 0.8 expected to make it into extras soon?

Comment 8 Shahms E. King 2006-05-11 14:12:38 UTC
Packages compiled yesterday should be in today's push.

Comment 9 Max Kanat-Alexander 2010-01-11 22:12:50 UTC
Could we get an update to bzr 1.18 or 2.0.4?

2.0.4 is a major update that changes the default repository format. 1.18 can *read* that format, but it doesn't perform very well myself.