Closed
Bug 68631
Opened 24 years ago
Closed 23 years ago
lack of arch-specific compiler flags in security/coreconf/Linux.mk
Categories
(Core Graveyard :: Security: UI, defect, P3)
Tracking
(Not tracked)
VERIFIED
INVALID
Future
People
(Reporter: antitux, Assigned: ddrinan0264)
References
()
Details
Attachments
(1 file)
752 bytes,
patch
|
Details | Diff | Splinter Review |
I did the official build of mozilla 0.7 for linux-sparc64, and came across a peculiar problem. Originally, there were no arch-specific compile flags for PSM in linux-sparc/sparc64, so when trying to make the official build, PSM wouldn't compile. I submitted a patch, which is still on Duke34.mcom.com, and was emailed back told that the problem was already solved. I think that the that was went about solving it was flawed. My reasoning behind this is that instead of adding arch-specific compile flags, you added generic ones, causing PSM to run at approximately 25% slower speed. I'm putting another patch on http://Duke34.mcom.com to show how I believe that the original patch SHOULD have been done. This might also solve you issues with compiles of PSM, if you're still looking at those. The new patch is located at http://duke34.mcom.com/architecture-compile-optimizations-Linux.mk.PATCH If you have any problems/comments/concerns, please contact me. Thanks
Reporter | ||
Comment 1•24 years ago
|
||
Comment 2•24 years ago
|
||
Marking NEW to get someone to look and review the patch.
Reporter | ||
Comment 3•24 years ago
|
||
I bass-ackwarded the patch, the fixed one is on http://bofh1.mcom.com
Comment 4•23 years ago
|
||
Reporter, is this still a problem now that PSM 2.0 is integrated into the nightly builds?
Comment 6•23 years ago
|
||
version 2.0 target 2.1
Target Milestone: 2.0 → 2.1
Version: unspecified → 2.0
Updated•23 years ago
|
Keywords: nsenterprise
Comment 7•23 years ago
|
||
Moving all P3 and P4 bugs targetted to 2.1 to future.
Target Milestone: 2.1 → Future
Comment 8•23 years ago
|
||
removing nsenterprise keyword from PSM bugs with target milestone of future.
Keywords: nsenterprise
Comment 10•23 years ago
|
||
Marking invalid, since there has been no comment from the reporter in 6 months. Reporter, please reopen if there is still a problem.
Status: NEW → RESOLVED
Closed: 23 years ago
QA Contact: ckritzer → junruh
Resolution: --- → INVALID
Reporter | ||
Comment 11•23 years ago
|
||
nah, that's cool.
Updated•8 years ago
|
Product: Core → Core Graveyard
You need to log in
before you can comment on or make changes to this bug.
Description
•