Closed
Bug 627277
Opened 14 years ago
Closed 14 years ago
rm -r widget/src/beos
Categories
(Core Graveyard :: Widget: BeOS, defect)
Tracking
(Not tracked)
RESOLVED
FIXED
Future
People
(Reporter: khuey, Assigned: Mitch)
References
Details
(Whiteboard: fixed-in-bs)
Attachments
(18 files, 3 obsolete files)
304.16 KB,
patch
|
Biesinger
:
review+
|
Details | Diff | Splinter Review |
10.98 KB,
patch
|
Gavin
:
review+
|
Details | Diff | Splinter Review |
19.13 KB,
patch
|
Biesinger
:
review+
|
Details | Diff | Splinter Review |
30.27 KB,
patch
|
roc
:
review+
Biesinger
:
review+
|
Details | Diff | Splinter Review |
15.00 KB,
patch
|
jorendorff
:
review+
|
Details | Diff | Splinter Review |
41.07 KB,
patch
|
shaver
:
review+
|
Details | Diff | Splinter Review |
30.36 KB,
patch
|
neil
:
review+
|
Details | Diff | Splinter Review |
2.78 KB,
patch
|
Biesinger
:
review+
|
Details | Diff | Splinter Review |
8.08 KB,
patch
|
smontagu
:
review+
|
Details | Diff | Splinter Review |
20.56 KB,
patch
|
joe
:
review+
|
Details | Diff | Splinter Review |
5.45 KB,
patch
|
dwitte
:
review+
|
Details | Diff | Splinter Review |
3.91 KB,
patch
|
shaver
:
review+
|
Details | Diff | Splinter Review |
10.10 KB,
patch
|
jaas
:
review+
|
Details | Diff | Splinter Review |
3.54 KB,
patch
|
taras.mozilla
:
review+
|
Details | Diff | Splinter Review |
7.14 KB,
patch
|
axel
:
review+
|
Details | Diff | Splinter Review |
3.06 KB,
patch
|
mfinkle
:
review+
|
Details | Diff | Splinter Review |
865 bytes,
patch
|
roc
:
review+
|
Details | Diff | Splinter Review |
11.08 KB,
patch
|
khuey
:
review+
|
Details | Diff | Splinter Review |
After we ship 2.0, I'd like to rm -r widget/src/beos. Roc owns widget/ and biesi is the owner of record for widget/src/beos, so CCing them both in case they object for some reason.
Comment 1•14 years ago
|
||
Yeah, realistically, this code isn't working now and probably won't in the future. People can revive it from the hg history if they want to work on it again.
Comment 2•14 years ago
|
||
After this lands, we should maybe WONTFIX all remaining bugs with [beos] in the title or platform fields.
WFM.
Comment 4•14 years ago
|
||
Zack, I just happened to notice your tryserver push, where you're moving some
files. Could you please not do that. Hg bug http://mercurial.selenic.com/bts/issue1576 should be fixed before file moves :(
Assignee | ||
Updated•14 years ago
|
Assignee: nobody → mitchell.field
Status: NEW → ASSIGNED
Version: unspecified → Trunk
Assignee | ||
Comment 5•14 years ago
|
||
Attachment #507430 -
Flags: review?(cbiesinger)
Assignee | ||
Comment 6•14 years ago
|
||
Attachment #507436 -
Flags: review?(gavin.sharp)
Assignee | ||
Comment 7•14 years ago
|
||
Attachment #507437 -
Flags: review?(jst)
Assignee | ||
Comment 8•14 years ago
|
||
Attachment #507442 -
Flags: review?(roc)
Assignee | ||
Comment 9•14 years ago
|
||
Attachment #507443 -
Flags: review?(roc)
Assignee | ||
Comment 10•14 years ago
|
||
Comment on attachment 507443 [details] [diff] [review]
layout
Disregard this. Other changes crept in.
Attachment #507443 -
Attachment is obsolete: true
Attachment #507443 -
Flags: review?(roc)
Assignee | ||
Comment 11•14 years ago
|
||
Attachment #507450 -
Flags: review?(jorendorff)
Assignee | ||
Comment 12•14 years ago
|
||
Attachment #507453 -
Flags: review?(shaver)
Assignee | ||
Comment 13•14 years ago
|
||
Attachment #507456 -
Flags: review?(neil)
Assignee | ||
Comment 14•14 years ago
|
||
Attachment #507458 -
Flags: review?(cbiesinger)
Updated•14 years ago
|
Attachment #507456 -
Flags: review?(neil) → review+
Assignee | ||
Comment 15•14 years ago
|
||
Attachment #507463 -
Flags: review?(smontagu)
Assignee | ||
Comment 16•14 years ago
|
||
Attachment #507467 -
Flags: review?(joe)
Assignee | ||
Comment 17•14 years ago
|
||
Attachment #507470 -
Flags: review?
Assignee | ||
Updated•14 years ago
|
Attachment #507470 -
Flags: review? → review?(dwitte)
Assignee | ||
Comment 18•14 years ago
|
||
Attachment #507474 -
Flags: review?(joshmoz)
Comment 19•14 years ago
|
||
Comment on attachment 507474 [details] [diff] [review]
plugin
>diff --git a/modules/plugin/base/src/Makefile.in b/modules/plugin/base/src/Makefile.in
>-ifeq ($(OS_ARCH), BeOS)
>- CPPSRCS += nsPluginsDirBeOS.cpp
>- CPPSRCS += nsPluginNativeWindow.cpp
>-else
> ifneq (,$(filter WINNT WINCE,$(OS_ARCH)))
> CPPSRCS += nsPluginsDirWin.cpp
I think you're forgetting to remove the matching "endif".
Attachment #507474 -
Flags: review?(joshmoz) → review-
Assignee | ||
Comment 20•14 years ago
|
||
No-one cares about Mork so I chose a reviewer for Firefox in general.
Attachment #507484 -
Flags: review?(shaver)
Assignee | ||
Comment 21•14 years ago
|
||
Attachment #507474 -
Attachment is obsolete: true
Attachment #507490 -
Flags: review?(joshmoz)
Attachment #507490 -
Flags: review?(joshmoz) → review+
Updated•14 years ago
|
Attachment #507436 -
Flags: review?(gavin.sharp) → review+
Assignee | ||
Comment 22•14 years ago
|
||
Attachment #507497 -
Flags: review?(tglek)
Updated•14 years ago
|
Attachment #507430 -
Flags: review?(cbiesinger) → review+
Updated•14 years ago
|
Attachment #507458 -
Flags: review?(cbiesinger) → review+
Assignee | ||
Updated•14 years ago
|
Attachment #507484 -
Flags: review?(shaver) → review?(gavin.sharp)
Updated•14 years ago
|
Attachment #507442 -
Flags: review+
Updated•14 years ago
|
Attachment #507437 -
Flags: review?(jst) → review+
Assignee | ||
Comment 23•14 years ago
|
||
Attachment #507502 -
Flags: review?(axel)
Updated•14 years ago
|
Attachment #507502 -
Flags: review?(axel) → review+
Assignee | ||
Comment 24•14 years ago
|
||
Attachment #507506 -
Flags: review?
Reporter | ||
Updated•14 years ago
|
Attachment #507506 -
Flags: review? → review?(gerv)
Assignee | ||
Comment 25•14 years ago
|
||
Attachment #507512 -
Flags: review?(mark.finkle)
Updated•14 years ago
|
Attachment #507512 -
Flags: review?(mark.finkle) → review+
Updated•14 years ago
|
Attachment #507497 -
Flags: review?(tglek) → review+
Assignee | ||
Comment 26•14 years ago
|
||
Attachment #507516 -
Flags: review?(roc)
Assignee | ||
Comment 27•14 years ago
|
||
Attachment #507517 -
Flags: review?(khuey)
Comment 28•14 years ago
|
||
Comment on attachment 507484 [details] [diff] [review]
db
Can we just remove mork entirely, or move it to Thunderbird? I guess that's for another bug.
Attachment #507484 -
Flags: review?(gavin.sharp) → review+
Reporter | ||
Comment 29•14 years ago
|
||
We still link morkreader into Firefox. After 2.0 ships I'll revive my patch to remove that and then we can move mork elsewhere.
Assignee | ||
Comment 30•14 years ago
|
||
Comment on attachment 507506 [details] [diff] [review]
relic
Files affected by this patch have coincidentally been deleted from mozilla-central.
Attachment #507506 -
Attachment is obsolete: true
Attachment #507506 -
Flags: review?(gerv)
Attachment #507442 -
Flags: review?(roc) → review+
Attachment #507516 -
Flags: review?(roc) → review+
Comment 31•14 years ago
|
||
Comment on attachment 507470 [details] [diff] [review]
libpref
r=dwitte
Attachment #507470 -
Flags: review?(dwitte) → review+
Reporter | ||
Comment 32•14 years ago
|
||
Comment on attachment 507517 [details] [diff] [review]
config
Presumably there should be an autoconf.mk.in removal for the AC_SUBST you're removing?
Attachment #507517 -
Flags: review?(khuey) → review+
Updated•14 years ago
|
Attachment #507450 -
Flags: review?(jorendorff) → review+
Comment 33•14 years ago
|
||
Comment on attachment 507467 [details] [diff] [review]
libpr0n
r=me presuming it passes try
Attachment #507467 -
Flags: review?(joe) → review+
Updated•14 years ago
|
Attachment #507463 -
Flags: review?(smontagu) → review+
Comment 34•14 years ago
|
||
Comment on attachment 507453 [details] [diff] [review]
xpcom
begone.
Attachment #507453 -
Flags: review?(shaver) → review+
Assignee | ||
Comment 35•14 years ago
|
||
http://hg.mozilla.org/projects/build-system/rev/1feb4a7c7ca6
http://hg.mozilla.org/projects/build-system/rev/347d38aeaacc
http://hg.mozilla.org/projects/build-system/rev/cd4e66877044
http://hg.mozilla.org/projects/build-system/rev/95f3fcd24cc4
http://hg.mozilla.org/projects/build-system/rev/c05a09d2592e
Assignee | ||
Updated•14 years ago
|
Whiteboard: fixed-in-bs
Reporter | ||
Comment 36•14 years ago
|
||
http://hg.mozilla.org/mozilla-central/rev/1feb4a7c7ca6
http://hg.mozilla.org/mozilla-central/rev/347d38aeaacc
http://hg.mozilla.org/mozilla-central/rev/cd4e66877044
http://hg.mozilla.org/mozilla-central/rev/95f3fcd24cc4
http://hg.mozilla.org/mozilla-central/rev/c05a09d2592e
Status: ASSIGNED → RESOLVED
Closed: 14 years ago
Resolution: --- → FIXED
Updated•10 years ago
|
Product: Core → Core Graveyard
You need to log in
before you can comment on or make changes to this bug.
Description
•