[OS/2] pyxpt doesn't work - no python mmap module found

RESOLVED FIXED in mozilla6

Status

()

Core
Build Config
--
major
RESOLVED FIXED
6 years ago
6 years ago

People

(Reporter: Walter Meinl, Assigned: glandium)

Tracking

Trunk
mozilla6
x86
OS/2
Points:
---

Firefox Tracking Flags

(Not tracked)

Details

Attachments

(1 attachment)

(Reporter)

Description

6 years ago
User-Agent:       Mozilla/5.0 (OS/2; Warp 4.5; rv:2.0.1) Gecko/20100101 Firefox/4.0.1
Build Identifier: 

E:/python26/python.exe ../../dist/sdk/bin/xpt.py link _xpidlgen/xpcom_base.xpt _xpidlgen/nsIConsoleListener.xpt _xpidlgen/nsIConsoleMessage.xpt _xpidlgen/nsIConsoleService.xpt _xpidlgen/nsICycleCollectorListener.xpt _xpidlgen/nsIDebug2.xpt _xpidlgen/nsIErrorService.xpt _xpidlgen/nsIException.xpt _xpidlgen/nsIExceptionService.xpt _xpidlgen/nsIVersionComparator.xpt _xpidlgen/nsIUUIDGenerator.xpt _xpidlgen/nsIMutable.xpt _xpidlgen/nsIMemoryReporter.xpt _xpidlgen/nsIDebug.xpt _xpidlgen/nsIInterfaceRequestor.xpt _xpidlgen/nsIMemory.xpt _xpidlgen/nsIProgrammingLanguage.xpt _xpidlgen/nsISupports.xpt _xpidlgen/nsITraceRefcnt.xpt _xpidlgen/nsIWeakReference.xpt _xpidlgen/nsrootidl.xpt
Traceback (most recent call last):
  File "../../dist/sdk/bin/xpt.py", line 69, in <module>
    import mmap
ImportError: No module named mmap
make.exe[6]: *** [_xpidlgen/xpcom_base.xpt] Error 1


Reproducible: Always
(Reporter)

Updated

6 years ago
Blocks: 643817
Version: unspecified → Trunk
Created attachment 527740 [details] [diff] [review]
possible patch

This passes the unit tests, but could you check if that works properly with a full build?
(Reporter)

Comment 2

6 years ago
(In reply to comment #1)
> Created attachment 527740 [details] [diff] [review]
> possible patch
> 
> This passes the unit tests, but could you check if that works properly with a
> full build?

Yep, this patch works for building on OS/2 and linux.
Attachment #527740 - Flags: review?(ted.mielczarek)
Comment on attachment 527740 [details] [diff] [review]
possible patch

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

I guess this is fine. I liked the conceptual purity of the mmap, but in practice XPT files are small so it doesn't matter much.
Attachment #527740 - Flags: review?(ted.mielczarek) → review+
Assignee: nobody → mh+mozilla
http://hg.mozilla.org/mozilla-central/rev/eebcd14c9c46

ted, do you want this landed on http://hg.mozilla.org/users/tmielczarek_mozilla.com/pyxpt/, too ?
Status: NEW → RESOLVED
Last Resolved: 6 years ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla6
Yeah, might as well keep them in sync for now.
Pushed http://hg.mozilla.org/users/tmielczarek_mozilla.com/pyxpt/rev/3b0d14d5563a
You need to log in before you can comment on or make changes to this bug.