38191 - tinderbox problems with multiple modules / branches (can't see NSPR checkins)

Status: RESOLVED WONTFIX

(Webtools Graveyard :: Tinderbox, defect, P3)

Description

[(not reading, please use [e-mail] instead)]

right now, tinderbox is set up to show checkins to the SeaMonkeyAll module

client.mak and client.mk pull more than that module to build mozilla.

right now, we pull nsprpub (with a static tag) but mozilla/security with a 
branch tag.

eventually (say after 6.0), we'll go to branch tags for layout, xpfe and rdf 
(and more, I'm sure) to cut down on day-to-day regressions.

before we do that, we need to identify and solve the tinderbox, lxr, bonsai 
problems with multiple modules / branches.

Comment 1

[Daniel (Leaf) Nunes]

JJ and i would just like to chime in and say ``this would be great!''

Maybe i'll do some real work and look at the tinderbox sources to fix; is the
rewrite of tinderbox going to address this?

Comment 2

[Chris McAfee]

m20

Comment 3

[Dawn Endico]

slamm no longer owns tinderbox and is no longer at netscape. Reassigning
to mcafee till a more permenant owner is found so these bugs won't be
ownerless.

Comment 4

[Dawn Endico]

Unsetting target milestone so we can delete the old (inappropriate)
Mozilla target milestones from Webtools.

Comment 5

[cls]

Attachment: Prelim support to tinderbox for multiple branches (untested)

Comment 6

[cls]

Attachment: Use branch specified in checkin data rather than one passed into cvsquery.cgi

Comment 7

[cls]

Ok, here's the prelim patches.  Tinderbox bits aren't really tested as I'm too
tired to setup clients and multiple branches for my tinderbox server.  Aside
from those patches, you will need to re-add the trees to the tinderbox server
with the branches that you want to look for in a space separated list.  Also, we
are going to need to define a TinderboxSeaMonkeyAll module to cvs that contains
*all* of the files that we will want to track (NSPR, gfx2, etc).  The changes
still assume that all of the files can be found in a single cvs module.

Comment 8

[cls]

Attachment: Updated tested patch for tinderbox

Comment 9

[Daniel (Leaf) Nunes]

cc'ing cyeh... i don't have an installation to test this on =/

Comment 10

[cls]

Reassigning to dawn.

Comment 11

[timeless]

mcafee: i'd like to up the priority of this bug, any objections?

Comment 12

[Chris McAfee]

timeless: this is endico's bug.

I haven't looked at these changes, maybe seawood can summarize
what happened with his changes so far.

Comment 13

[hacker formerly known as [e-mail]]

Comment on attachment 27155 [details] [diff] [review]
Updated tested patch for tinderbox

Executive summary: They didn't work.

Comment 15

[Dave Miller [:justdave]]

Did we just fix this with the adding of the SecurityServices module?

Comment 16

[Nelson Bolyard (seldom reads bugmail)]

I think not  (said Rene Descartes just before he vanished :)

SecurityServices allows us to have a single page that can see changes to 
multiple modules, as long as all changes are on the trunk or on the same
named branch in all modules.  But of course, each module uses its own 
branch names, so in practice this solution only works for the trunk.

I think Seth wanted a way to be able to specify multiple modules, each 
with its own separate branch tag (or HEAD for the trunk), so that a 
single tinderbox page would show checkins to any of those modules on 
their respective branches.   

The SecurityServices teams wants that, too, very much.  The recent creation
of the SecurityServices module was just a short term workaround, and partial
one at that.  

Comment 17

[Sylvestre Ledru [:Sylvestre]]

Tinderbox isn't maintained anymore. Closing.