CVS account request Dirk Mueller for Firefox/Qt

RESOLVED INCOMPLETE

Status

mozilla.org
Repository Account Requests
RESOLVED INCOMPLETE
12 years ago
8 years ago

People

(Reporter: Dirk Mueller, Assigned: marcia)

Tracking

Details

Attachments

(1 attachment)

229 bytes, text/plain
Details
(Reporter)

Description

12 years ago
User-Agent:       Mozilla/5.0 (compatible; Konqueror/3.4; Linux) KHTML/3.4.89 (like Gecko)
Build Identifier: 

I'd like to request a CVS account for improving the Qt toolkit port of Mozilla 
in cooperation with Zack Rusin <zack@kde.org>.  
 
 

Reproducible: Always
(Reporter)

Comment 1

12 years ago
Created attachment 186330 [details]
ssh key

Comment 2

12 years ago
Dirk's one of the best KDE developers. He's got tremendous experience with
writing browsers and is a very reliable engineer. 
I'm vouching for Dirk.
(Assignee)

Comment 3

12 years ago
Need the CVS form  - www.mozilla.org/hacking/form.html
Status: UNCONFIRMED → ASSIGNED
Ever confirmed: true
(Reporter)

Comment 4

12 years ago
FAXed the form. 
(Reporter)

Comment 5

12 years ago
I don't think I need super-reviewal because I'll be only touching 
things inside the qt/* subdirs.  
 
 
So you're not going to touch the UI bits that control button ordering/theme
tweaks or anything else outside of the widget code?  I really doubt that'd last,
unless KDE is going to switch to GNOME-style bits in dialogs, etc.
(Reporter)

Comment 7

12 years ago
currently some of the dialogs are natively implemented in Qt, so they have the 
"right" button ordering already. If we're there that I'll have time to care 
about those issues, then I'll ask for review.  
If the SR requirement is waived, then you're expected to restrict all checkins
to the specific area you'll be working within.  In this case, I think its
probably appropriate/desirable to get the three SRs so as to not have to worry
about restrictions.  The exemption is really more intended for NSS, Chatzilla,
and other localized and tightly-owned pieces that don't spill over into core code.

Not trying to be a hardass, just trying to clarify.
Dirk, have you already done any patches, by chance?  Or are there bugs where you
have patches posted right now?
I don't think Dirk has done much yet. But Wolfgang and I can handle his first
few non-Qt checkins and then he can get general CVS access.
That sounds pretty reasonable to me.
QA Contact: justdave → justdave

Comment 12

12 years ago
In terms of reducing bureaucracity, mozilla can certainly learn from kde...
what is blocking us in this bug?
Can we see some representative patches (bug numbers)? Reading the bug it doesn't look like drivers have agreed to waive the three-superreview rule here (yet).
(In reply to comment #12)
> In terms of reducing bureaucracity, mozilla can certainly learn from kde...

Why should we give cvs access to people who haven't yet written any patches to Mozilla?
> what is blocking us in this bug?

The fact that we give CVS access to people who've shown they write good code and follow Mozilla procedures.  And Dirk, not having written any code that I know of, hasn't shown that.
Summary: CVS acccount request Dirk Mueller for Firefox/Qt → CVS account request Dirk Mueller for Firefox/Qt
QA Contact: justdave → cvs-acct-req
Due to a recent overhaul of the committer's agreement, we are requiring new forms be sent for accounts that have yet to be created, even if you have already filled out and submitted the old CVS Contributor Form. Please download the form at http://www.mozilla.org/hacking/committer/committers-agreement.pdf and submit it according to the directions specified at http://www.mozilla.org/hacking/notification/. We apologize for this necessary inconvenience. If you have any questions, feel free to ask. Thanks!
This request has been inactive for a long time, so closing it out to get it off
the list of open bugs. Feel free to reopen or file a new bug if you're still
interested in access.
Status: ASSIGNED → RESOLVED
Last Resolved: 8 years ago
Resolution: --- → INCOMPLETE
You need to log in before you can comment on or make changes to this bug.