Closed
Bug 87698
Opened 23 years ago
Closed 23 years ago
give rogc@netscape.com CVS access
Categories
(mozilla.org :: Repository Account Requests, task)
mozilla.org
Repository Account Requests
Tracking
(Not tracked)
RESOLVED
FIXED
People
(Reporter: waterson, Assigned: marcia)
Details
Let's land the mothership and get him on board. Give it up! Unh unh! Give it up!
Comment 1•23 years ago
|
||
Who all will sr=? I'm guessing myself, mebbe shaver cuz of the grope
connection. Waterson is voucher, we need a third non-netscape.com, non-vouching
sr= (toldya to get cathleen to vouch! silly rules... seriously, we can do this
right and not add delay, I think).
/be
Assignee | ||
Comment 2•23 years ago
|
||
Roger - You will need to fill out the CVS Contributor form
(http://www.mozilla.org/hacking/form.html) and get it back to me to start the
process.
Status: NEW → ASSIGNED
Comment 3•23 years ago
|
||
I faxed in the form.
Thanks!
-Roger
Assignee | ||
Comment 4•23 years ago
|
||
form received.
Comment 5•23 years ago
|
||
Do you wanna see more code that I've written?
There's a patch for the timeline service that I wrote attached
to bug 78793: http://bugzilla.mozilla.org/showattachment.cgi?attach_id=33241
You'll get lots of conflicts if you try to apply this. I want to
point out the code in nsITimelineService.idl, nsTimelineService.cpp,
and nsTimelineService.h, which I think demonstrates a good level of
understanding of XPCOM (but hey, maybe I'm wrong :-).
The patch also demonstrates that I'm not fully up to speed on making
patches, because if you run "patch" on this beast the TimelineService
files don't end up in the right place.
If I were an sr for this code, I would ask why it's got its own
stopwatch-like service when there are already two of them in the
code base. The reason is that neither of the ones that are already
there do quite what I want. Before checking this code in I would
explore collapsing them all down into one.
The patch only covers Windows, not Linux. For the Linux build I'd
add a new configure option for the timeline. To be honest, I am
completely clueless on the Mac. I wouldn't check this in without
getting help on the Mac. The good news is that I have a Mac so once
I'm up to speed I won't drain other people's time doing Mac work
for me.
Here's a patch from bug 46776 that Cathleen checked in for me that
demonstrates competence with the Linux build system (or so I hope :-):
http://bugzilla.mozilla.org/showattachment.cgi?attach_id=39024
This patch adds a --enable-reorder config option for function reordering
on Linux. Again demonstrating my lack of competence with generating
patches, the new files that were needed for this change were attached
to the bug as a zip file:
http://bugzilla.mozilla.org/showattachment.cgi?attach_id=39025
Finally, I attached the source to the tools I've been using for
reordering to bug 65845:
http://bugzilla.mozilla.org/showattachment.cgi?attach_id=38191
This code is intentionally *not* mozilla-style code because it's used
to instrument mozilla and I didn't want any circular dependencies.
It uses STL because I was hacking this together quickly; I would have
grave concerns about using STL in a shipping product. What you can
get out of this is demonstration of knowledge about sockets and C++.
So... is this enough for me to get my secret decoder ring (as Brendan
would say :-)?
Thanks!
-Roger
Comment 6•23 years ago
|
||
I'm sr'ing access for Roger. Cc'ing scc in the hope that he can take a look and
vouch too. Shaver, ping.
/be
Comment 7•23 years ago
|
||
His work looks good, though we need to bring him over the world of the _M_PL for
licenses. I'll vouch. sr=scc.
Assignee | ||
Comment 8•23 years ago
|
||
okay, roger is only 1 sr away from access. hopefully shaver will chime in soon
so i can flip the switch.
sr=shaver. Flip his switch!
Assignee | ||
Updated•23 years ago
|
Status: ASSIGNED → RESOLVED
Closed: 23 years ago
Resolution: --- → FIXED
Assignee | ||
Comment 10•23 years ago
|
||
mothership has landed. account activated.
You need to log in
before you can comment on or make changes to this bug.
Description
•