Bug 544628

Summary: at-spi2-core - Protocol definitions and daemon for D-Bus at-spi
Product: [Fedora] Fedora Reporter: Matthias Clasen <mclasen>
Component: Package ReviewAssignee: Thomas Spura <tomspur>
Status: CLOSED RAWHIDE QA Contact: Fedora Extras Quality Assurance <extras-qa>
Severity: medium Docs Contact:
Priority: low    
Version: rawhideCC: ben, fedora-package-review, notting, tomspur, zcerza
Target Milestone: ---Flags: tomspur: fedora‑review+
dennis: fedora‑cvs+
Target Release: ---   
Hardware: All   
OS: Linux   
Whiteboard:
Fixed In Version: Doc Type: Bug Fix
Doc Text:
Story Points: ---
Clone Of: Environment:
Last Closed: 2010-01-04 21:27:17 EST Type: ---
Regression: --- Mount Type: ---
Documentation: --- CRM:
Verified Versions: Category: ---
oVirt Team: --- RHEL 7.3 requirements from Atomic Host:
Cloudforms Team: ---
Bug Depends On:    
Bug Blocks: 544629, 544630    

Description Matthias Clasen 2009-12-05 14:34:47 EST
The accessibility framework is being rewritten on top of D-Bus to get rid of ORBit/CORBA. This package is an early release of the core of the new framework.

I'm packaging it now to allow people to start testing and reporting problems. 
For now, the at-spi2 packages will have Conflicts with the at-spi packages. This
will be replaced by Provides/Obsoletes, once at-spi2 is feature-complete and ready for prime time.

SRPM: http://mclasen.fedorapeople.org/at-spi2/at-spi2-core-0.1.3-1.fc12.src.rpm
Spec: http://mclasen.fedorapeople.org/at-spi2/at-spi2-core.spec

Mock complains about three empty interface definitions, which is probably a problem with the xslt script that converts them from the docs. I'll report that upstream.
Comment 1 Thomas Spura 2009-12-05 16:35:59 EST
Just a few comments:

- There is no conflicts with the at-spi packages as described above, yet.
- source url is wrong: should be download*DOT*gnome*DOT*org
Comment 2 Matthias Clasen 2009-12-06 11:03:56 EST
This one is not conflicting. The at-spi2-atk and pyatspi packages are, though.
Comment 3 Thomas Spura 2009-12-21 07:37:37 EST
Review:

Good:
- license ok
- name ok
- buildroot ok (unneeded in fedora)
- parallel make
- %clean section there
- %files section ok
  * owns all directories it creates
  * requires d-bus for one folder
- buildrequires/requires ok
- builds in koji
  http://koji.fedoraproject.org/koji/taskinfo?taskID=1882908
- noarch not possible
- no *.la files
- no shared libs
- %doc ok


Needswork:
- URL still wrong, see comment #1
- from https://fedoraproject.org/wiki/Packaging/Guidelines#Libexecdir
  'Packagers are highly encouraged to store libexecdir files in a package-specific subdirectory of %{_libexecdir}, such as %{_libexecdir}/%{name}'
- What's the status of the xslt script?



############################

Please fix the url; subdirectory of %{_libexecdir} and 'Is programm working as expected?' are just 'should' items.

############################

APPROVED
Comment 4 Matthias Clasen 2009-12-22 19:49:04 EST
Thanks for the review. 
I've fixed the url in the package that I am going to import..
Comment 5 Matthias Clasen 2009-12-22 19:50:31 EST
New Package CVS Request
=======================
Package Name: at-spi2-core
Short Description: Protocol definitions and daemon for D-Bus at-spi
Owners: mclasen
Branches: 
InitialCC:
Comment 6 Dennis Gilmore 2009-12-23 14:51:22 EST
CVS Done
Comment 7 Matthias Clasen 2010-01-04 21:27:17 EST
Build done.