[Bug 168210] Review Request: phpldapadmin - Web-based tool for managing LDAP servers

[bugzilla at redhat.com]

Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug report.

Summary: Review Request: phpldapadmin - Web-based tool for managing LDAP servers


https://bugzilla.redhat.com/bugzilla/show_bug.cgi?id=168210





------- Additional Comments From gauret at free.fr  2005-09-26 13:10 EST -------
I'd prefer not to include those patches in the rpm, and just wait until they are
included upstream. That's Fedora's policy in general (especially true when
applied to the kernel). I understand that those are your patches, and submitting
them upstream was the right move. Let's just wait for them to be official, we're
bleeding-edge enough.

The problem with the debug_log function pruning is that I don't want to see this
kind of scenario in upstream's bug tracker :
"Please send your logs. Ah, you're using Fedora ? Well, CLOSED WONTFIX".
If the issue is posted upstream too, couldn't we wait for it to be resolved
there ? It would probably be done in a cleaner way (eg: put the sprintf lower on
the stack, and keep the debug)
For regular packages, we have the -debuginfo rpm if we need debug. Here, the
debug is gone for good.

IMHO, patches should almost only be used for Fedora-specific customizations.

Concerning the devel branch, that's not a problem, just request branches when
you judge it stable enough.

-- 
Configure bugmail: https://bugzilla.redhat.com/bugzilla/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
You are on the CC list for the bug, or are watching someone who is.