[Bug 891312] New: Review Request: ldapdiff - Tool for incremental LDAP directory updates based on ldif files
8:54 a.m.
Product: Fedora
https://bugzilla.redhat.com/show_bug.cgi?id=891312
Bug ID: 891312
Summary: Review Request: ldapdiff - Tool for incremental LDAP
directory updates based on ldif files
Product: Fedora
Version: rawhide
Component: Package Review
Severity: medium
Priority: medium
Reporter: kwizart(a)gmail.com
Spec URL: http://kwizart.fedorapeople.org/review/ldapdiff.spec
SRPM URL: http://kwizart.fedorapeople.org/review/ldapdiff-1.4.1-1.fc17.src.rpm
Description: Tool for incremental LDAP directory updates based on ldif files
Fedora Account System Username: kwizart
Koji rawhide scratch build:
http://koji.fedoraproject.org/koji/taskinfo?taskID=4832975
rpmlint (from el6):
$ rpmlint ldapdiff-1.4.1-1.fc17.src.rpm
ldapdiff.src:35: E: files-attr-not-set
ldapdiff.src:36: E: files-attr-not-set
ldapdiff.src:37: E: files-attr-not-set
ldapdiff.src:38: E: files-attr-not-set
ldapdiff.src:39: E: files-attr-not-set
ldapdiff.src:40: E: files-attr-not-set
ldapdiff.src:41: E: files-attr-not-set
ldapdiff.src: W: no-cleaning-of-buildroot %clean
ldapdiff.src: W: no-buildroot-tag
ldapdiff.src: W: no-%clean-section
Should be safe to ignore
--
You are receiving this mail because:
You are on the CC list for the bug.
Unsubscribe from this bug
https://bugzilla.redhat.com/token.cgi?t=42pgACxZSi&a=cc_unsubscribe
9:05 a.m.
New subject: [Bug 891312] Review Request: ldapdiff - Tool for incremental LDAP directory updates based on ldif files
Product: Fedora
https://bugzilla.redhat.com/show_bug.cgi?id=891312
Mamoru TASAKA <mtasaka(a)fedoraproject.org> changed:
What |Removed |Added
----------------------------------------------------------------------------
Status|NEW |ASSIGNED
Assignee|nobody(a)fedoraproject.org |mtasaka(a)fedoraproject.org
Flags| |fedora-review?
--- Comment #1 from Mamoru TASAKA <mtasaka(a)fedoraproject.org> ---
Taking this. I would appreciate it if you would review my review request bug
875352.
--
You are receiving this mail because:
You are on the CC list for the bug.
Unsubscribe from this bug
https://bugzilla.redhat.com/token.cgi?t=BjSBckLXpf&a=cc_unsubscribe
9:37 a.m.
New subject: [Bug 891312] Review Request: ldapdiff - Tool for incremental LDAP directory updates based on ldif files
Product: Fedora
https://bugzilla.redhat.com/show_bug.cgi?id=891312
--- Comment #2 from Nicolas Chauvet (kwizart) <kwizart(a)gmail.com> ---
(In reply to comment #1)
Taking this. I would appreciate it if you would review my review
request bug
875352.
Sure, I will handle when back to home.
--
You are receiving this mail because:
You are on the CC list for the bug.
Unsubscribe from this bug
https://bugzilla.redhat.com/token.cgi?t=5zKImhVU0M&a=cc_unsubscribe
10:13 a.m.
New subject: [Bug 891312] Review Request: ldapdiff - Tool for incremental LDAP directory updates based on ldif files
Product: Fedora
https://bugzilla.redhat.com/show_bug.cgi?id=891312
Mamoru TASAKA <mtasaka(a)fedoraproject.org> changed:
What |Removed |Added
----------------------------------------------------------------------------
Flags|fedora-review? |
Flags| |fedora-review+
--- Comment #3 from Mamoru TASAKA <mtasaka(a)fedoraproject.org> ---
Well, while I have not checked in detail how ldapdiff utility
works, packaging for this software seems simple and reviewing
for this srpm is straightforward.
Some minor issues (not blockers)
* Cleaning %buildroot at %install
https://fedoraproject.org/wiki/Packaging:Guidelines?rd=Packaging/Guidelin...
- "rm -rf $RPM_BUILD_ROOT"
at the beginning of %install is no longer needed
because rpmbuild will always clean up $RPM_BUILD_ROOT
at the beginning of %install.
Other things are okay
- spec file clean
- tarball matches with the upstream
- build clean
- built binary rpm clean
- can be installed
- at least ldapdiff -v works
----------------------------------------------------------
This package (ldapdiff) is APPROVED by mtasaka
----------------------------------------------------------
--
You are receiving this mail because:
You are on the CC list for the bug.
Unsubscribe from this bug
https://bugzilla.redhat.com/token.cgi?t=IbA4do9Rm4&a=cc_unsubscribe
5:29 p.m.
New subject: [Bug 891312] Review Request: ldapdiff - Tool for incremental LDAP directory updates based on ldif files
Product: Fedora
https://bugzilla.redhat.com/show_bug.cgi?id=891312
Nicolas Chauvet (kwizart) <kwizart(a)gmail.com> changed:
What |Removed |Added
----------------------------------------------------------------------------
Flags| |fedora-cvs?
--- Comment #4 from Nicolas Chauvet (kwizart) <kwizart(a)gmail.com> ---
New Package SCM Request
=======================
Package Name: ldapdiff
Short Description: Tool for incremental LDAP directory updates based on ldif
files
Owners: kwizart
Branches: f17 f18 el6 el5
InitialCC:
--
You are receiving this mail because:
You are on the CC list for the bug.
Unsubscribe from this bug
https://bugzilla.redhat.com/token.cgi?t=9BxJNNOFiV&a=cc_unsubscribe
9:21 a.m.
New subject: [Bug 891312] Review Request: ldapdiff - Tool for incremental LDAP directory updates based on ldif files
Product: Fedora
https://bugzilla.redhat.com/show_bug.cgi?id=891312
--- Comment #5 from Jon Ciesla <limburgher(a)gmail.com> ---
Git done (by process-git-requests).
--
You are receiving this mail because:
You are on the CC list for the bug.
Unsubscribe from this bug
https://bugzilla.redhat.com/token.cgi?t=uOzNOFRx6l&a=cc_unsubscribe
10:31 a.m.
New subject: [Bug 891312] Review Request: ldapdiff - Tool for incremental LDAP directory updates based on ldif files
Product: Fedora
https://bugzilla.redhat.com/show_bug.cgi?id=891312
Nicolas Chauvet (kwizart) <kwizart(a)gmail.com> changed:
What |Removed |Added
----------------------------------------------------------------------------
Status|ASSIGNED |CLOSED
Resolution|--- |NEXTRELEASE
Flags|fedora-cvs+ |
Flags| |fedora-cvs?
Last Closed| |2013-01-03 11:31:49
--- Comment #6 from Nicolas Chauvet (kwizart) <kwizart(a)gmail.com> ---
imported and built, thx!
--
You are receiving this mail because:
You are on the CC list for the bug.
Unsubscribe from this bug
https://bugzilla.redhat.com/token.cgi?t=vbB9rhFocp&a=cc_unsubscribe
10:47 a.m.
New subject: [Bug 891312] Review Request: ldapdiff - Tool for incremental LDAP directory updates based on ldif files
Product: Fedora
https://bugzilla.redhat.com/show_bug.cgi?id=891312
Mamoru TASAKA <mtasaka(a)fedoraproject.org> changed:
What |Removed |Added
----------------------------------------------------------------------------
Flags|fedora-cvs? |
--- Comment #7 from Mamoru TASAKA <mtasaka(a)fedoraproject.org> ---
(Just unsetting fedora-cvs flag: no longer needed)
--
You are receiving this mail because:
You are on the CC list for the bug.
Unsubscribe from this bug
https://bugzilla.redhat.com/token.cgi?t=hdReEaDpab&a=cc_unsubscribe
4156
days inactive
4157
days old
package-review@lists.fedoraproject.org
7 comments
1 participants
participants (1)
-
Red Hat Bugzilla