[Bug 513896] Review Request: pcp - performance monitoring and collection service

[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=513896





--- Comment #11 from Mark Goodwin <mgoodwin@xxxxxxxxxx>  2009-08-05 22:55:39 EDT ---

Hi Eric, I've just posted v3.0.0-2 :

Spec URL: ftp://oss.sgi.com/projects/pcp/download/v3/pcp.spec
Tar URL : ftp://oss.sgi.com/projects/pcp/download/v3/pcp-3.0.0-2.src.tar.gz
SRPM URL: ftp://oss.sgi.com/projects/pcp/download/v3/pcp-3.0.0-2.fc10.src.rpm

Merged with pcp-2.9.0-1 and pushed up to:
  git://oss.sgi.com/markgw/pcp/pcp.git dev

Changes since 3.0.0-1 are listed in the git logs, in summary :

    - merge with pcp-2.9.0-1 dev

    - bump to 3.0.0-2, update changelog and delete unncessary Vendor string

    - changes to the Fedora RPM spec in response to review feedback:
      - Incorporate a large number of suggestions and cleanups from Eric
Sandeen
      - Greatly simplified the %post and %postun scriptlets - remove ancient
crud
      - Added ldconfig scriptlets for -libs
      - Don't explicitly require IB libs, since they're libs and RPM figures it
out
      - No need to explicitly BuildRequire gcc-c++ libstdc++-devel
      - Add BuildRequires on perl-ExtUtils-MakeMaker
      - Remove explicit ia64 Requires: libunwind
      - Have pcp-libs require the base package since Fedora insists
      - Add and clarify some comments
      - Create %{_localstatedir}/run/pcp and ship it (so it'll be removed)
      - Use %doc for CHANGELOG COPYING INSTALL README VERSION.pcp pcp.lsm

    - git ignore generated gram*.tab.c

    - preserve generated gram.tab.c because debuginfo needs it

    - fix minor typo (Redhat -> Red Hat) in comment in Fedora spec

    - dont install trace demo binaries since src is installed anyway

    - tweak configure to move PCP_BINADM_DIR out of /usr/share, into /usr/lib.

    - Arch dependent binaries should not be installed below /usr/share.

    - delete unneeded explicit script interpreter to keep rpmlint happy

    - pmpost does not really need to be setuid

    - Nuke migrate_pcp_var_dir, no longer needed

    - default chkconfig off for all PCP services

Still have the (benign) rpmlint errors:
pcp.x86_64: E: subsys-not-used /etc/rc.d/init.d/pmproxy
pcp.x86_64: E: subsys-not-used /etc/rc.d/init.d/pmie
pcp.x86_64: E: subsys-not-used /etc/rc.d/init.d/pcp

Cheers
-- Mark Goodwin

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