Closed Bug 1089446 Opened 6 years ago Closed 6 years ago

Remove ancient and unused Perl scripts


(Core :: General, defect)

Not set





(Reporter: njn, Assigned: njn)




(6 files, 1 obsolete file)

By my count we currently have 94 Perl scripts in the repo. Some of them are imported third-party code, some of them are still useful, but some of them are unused. This bug is about eliminating the latter category. was added in 2001 as part of the "Neck [sic?] Page Load Timer".
It hasn't been touched since then, and I suspect it's unused.
Attachment #8511769 - Flags: review?(mcmanus)
roc says this is unused.
Attachment #8511770 - Flags: review?(roc)
build/unix/ is a script that prints its given arguments with duplicates
removed. Almost trivial, but implemented in surprisingly many lines of Perl.
(To be fair, it implements some options that we no longer need.)

This patch:

- Reimplements the script as a Python almost-one-liner.
- Removes the test, which IMO is unnecessary for something so simple. (The test
  was also surprisingly many lines.)

- Removes the corresponding test harness, which contained just this one test.

 8 files changed, 12 insertions(+), 385 deletions(-)


One thing to note is that doesn't preserve the original ordering, and I
think did. But that doesn't seem to matter.
Attachment #8511785 - Flags: review?(gps)
Comment on attachment 8511785 [details] [diff] [review]
Reimplement build/unix/ in Python, and remove its unit test

Review of attachment 8511785 [details] [diff] [review]:

Wow. Just wow.
Attachment #8511785 - Flags: review?(gps) → review+
> By my count we currently have 94 Perl scripts in the repo.

Er, make that 97.
Comment on attachment 8511806 [details] [diff] [review]
Improve the comments in both scripts

Attachment #8511806 - Flags: review?(bzbarsky) → review+
Comment on attachment 8511808 [details] [diff] [review]
Remove, which is ancient and unused

Attachment #8511808 - Flags: review?(bzbarsky) → review+
Attachment #8512261 - Flags: review?(mh+mozilla)
Depends on: 1089832
Depends on: 969985
Duplicate of this bug: 1089843
Comment on attachment 8512261 [details] [diff] [review]
Remove config/, which is ancient and unused

Your patch is empty, and it's also bug 1089855
Attachment #8512261 - Flags: review?(mh+mozilla) → review-
Depends on: 1089855
Attachment #8512261 - Attachment is obsolete: true
Attachment #8512259 - Flags: review?(nfroyd) → review+
Comment on attachment 8511769 [details] [diff] [review]
Remove netwerk/test/, which is ancient and unused

Review of attachment 8511769 [details] [diff] [review]:

please also remove PageList.txt which is the script's old input
Attachment #8511769 - Flags: review?(mcmanus) → review+
I over-counted initially. We're down to 88 now.
You need to log in before you can comment on or make changes to this bug.