[Bug 591305] Review Request: apache-commons-digester - XML to Java object mapping module

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

Stanislav Ochotnicky <sochotni@xxxxxxxxxx> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
             Status|NEW                         |ASSIGNED
         AssignedTo|nobody@xxxxxxxxxxxxxxxxx    |sochotni@xxxxxxxxxx

--- Comment #5 from Stanislav Ochotnicky <sochotni@xxxxxxxxxx> 2010-05-13 08:28:02 EDT ---
(In reply to comment #3)
> Sure, I can submit the apache-commons-beanutils package for review too, if
> you'd like?

That would be great. Just don't forget to cc current owner of
jakarta-commons-beanutils (ideally also add him as co-maintainer at the end of
review if he doesn't mind).


> (In reply to comment #2)
> > (since apache-commons-loggins is already in
> > rawhide)? Also you have old changelog included, it would probably be better to
> > start from scratch.
> > 
> 
> I have nothing against elderly changelogs, and it's not particularly long. I
> know packages like Kernel and Eclipse trim them periodically because they get
> absurdly long, but I thought it would be frowned upon to trim it for purely
> aesthetic reasons...

I wouldn't say it's aesthetic reason...You are creating a new package so if
anyone is looking for history of the older package they can check its history.
At least that's what I did with my commons packages: trimmed changelog, started
counting from revision 1. But this is just my preference, AFAIK there is no
policy forcing either way.


I'll take this for review once beanutils is in rawhide (so maybe I'll review
that first..)

-- 
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.
_______________________________________________
package-review mailing list
package-review@xxxxxxxxxxxxxxxxxxxxxxx
https://admin.fedoraproject.org/mailman/listinfo/package-review

[Index of Archives]     [Fedora Legacy]     [Fedora Desktop]     [Fedora SELinux]     [Yosemite News]     [KDE Users]     [Fedora Tools]