Closed
Bug 16718
Opened 25 years ago
Closed 25 years ago
rename files under mozilla/mailnews/news
Categories
(MailNews Core :: Backend, defect, P3)
Tracking
(Not tracked)
VERIFIED
WONTFIX
People
(Reporter: sspitzer, Assigned: sspitzer)
Details
under mozilla/mailnews/news/*
we have
nsNNTP*.cpp, nsNntp*.cpp and nsNews*.cpp
it'd be nice to just pick one. this would be a monster change, but would be
great.
Comment 1•25 years ago
|
||
carpool with leaf on this one - he can change the CVS files themselves, and
coordinate the unix makefile changes
Assignee | ||
Comment 2•25 years ago
|
||
the files aren't the big pain, it is the class name cleanup.
but nothing a good sed / perl script can't cure.
Comment 3•25 years ago
|
||
Actually the file name changes are the biggest part. Do to the fact that
CVS is case insensitive, you won't be able to rename nsNNTP* stuff to nsNntp*.
I already tried to fix this problem for news back in March and was unable to
do so because of this problem. You'll need to get Leaf to do it for you and
he gave me a hard time (in a very good natured way of course!) about doing it.
Comment 4•25 years ago
|
||
hey, CVS is case sensitive, it's windows that's not :)
Assignee | ||
Comment 5•25 years ago
|
||
accepting bug.
I don't see a problem with nsNews and nsNNTP both existing, for there is code in
the news world that isn't just NNTP-related. (Right?)
And when you're renaming, I _really_ _really_ think that we want to have
nsNNTPProtocol rather than nsNntpProtocol: NNTP is an acronym, and should be
capitalized as such. Don't make me come over there.
Assignee | ||
Comment 7•25 years ago
|
||
I'll do anything to prevent getting pimp slapped by shaver.
nsNews and nsNNTP is good with me.
this work will require leaf's help.
but don't worry leaf, we probably won't get to this for a while.
Comment 8•25 years ago
|
||
Bring it on, homeslice. I'm ready for you any time.
Assignee | ||
Updated•25 years ago
|
Target Milestone: M15
Assignee | ||
Comment 9•25 years ago
|
||
marking m15.
Assignee | ||
Updated•25 years ago
|
Target Milestone: M15 → M20
Assignee | ||
Comment 10•25 years ago
|
||
not going to happen any time soon.
moving to m20.
Assignee | ||
Comment 11•25 years ago
|
||
who are we kidding. I'm not going to get to this type of bug in 6.0
marking m30.
Target Milestone: M20 → M30
Assignee | ||
Comment 12•25 years ago
|
||
who are we kidding.
Status: ASSIGNED → RESOLVED
Closed: 25 years ago
Resolution: --- → WONTFIX
Updated•20 years ago
|
Product: MailNews → Core
Updated•16 years ago
|
Product: Core → MailNews Core
You need to log in
before you can comment on or make changes to this bug.
Description
•