Closed Bug 366355 Opened 16 years ago Closed 16 years ago

remove js/src/perlconnect

Categories

(Firefox Build System :: General, defect)

Product:
Firefox Build System
Component:
General
Type:
defect
Priority:
Not set
Severity:
normal

Tracking

(Not tracked)

Status:
RESOLVED FIXED

People

(Reporter: pavlov, Assigned: pavlov)

Details

Attachments

(1 file)

more perlconnect removal
2.05 KB, patch
mrbkap
: review+
Details | Diff | Splinter Review 
this code seems unowned and dead.  can we remove it?
I think Benjamin Livshits no longer cares, but if someone could double-check with him (timeless?).  Even if it's a hobby for one person (timeless?), it should be moved to sourceforge, or just removed.

/be
tagged with PERLCONNET_PRE_REMOVAL and cvs removed.
Status: NEW → RESOLVED
Closed: 16 years ago
Resolution: --- → FIXED
Removes some perlconnect left-overs.
Comment on attachment 266919 [details] [diff] [review]
more perlconnect removal

This looks good to me, but I'll feel happier if someone else takes a quick look.
Attachment #266919 - Flags: review?(mrbkap)
Comment on attachment 266919 [details] [diff] [review]
more perlconnect removal

Seems reasonable.
Attachment #266919 - Flags: review?(mrbkap) → review+
I will land (re-land, *blush*) this as soon as the tree re-opens.
Whiteboard: [needs checkin]
Whiteboard: [needs checkin] → [checkin needed]
Some time back I was involved in perlconnect a bit as we used this component internally with a standalone JS engine. We did some changes... Most changes was put back to the Mozilla tree, some perhaps not.

Just in case someone wants to play with it... feel free to contact me.
Makefile.ref: 3.45
js.c: 3.146
I take it there's no more [checkin needed], then?
Whiteboard: [checkin needed]
Product: Core → Firefox Build System
You need to log in before you can comment on or make changes to this bug.