390292 - Exports needed for SQLite on OS/2

Status: RESOLVED FIXED

(Toolkit :: Storage, defect)

Description

[Peter Weilbacher]

Shawn likes it better if one opens new bugs for OS/2 problems, so here we go. This is about the build break from bug 306907#c74. Will attach a new, hopefully better, patch in a moment.

Comment 1

[Peter Weilbacher]

Attachment: use windows .def file as base for OS/2 add_defs

If I reformat the Windows .def file a little I can easily use it as an input file to create the exports list for OS/2 (which needs an extra underscore prepended). I didn't actually find a guide for Windows .def files so I hope that like on OS/2 I can move the DATA directive to before the EXPORTS line.

Comment 2

[Shawn Wilsher :sdwilsh]

Comment on attachment 274618 [details] [diff] [review]
use windows .def file as base for OS/2 add_defs

I'm not the appropriate person to review this.  Probably bsmedberg.

Also, please get someone on windows to test this.

Comment 3

[Peter Weilbacher]

Michael, before I submit this to review again, you seem to be the one who added the
   sqlite3Os DATA
line to sqlite.def originally. Do you think moving the DATA around like in this patch will work or do you have another idea?

Comment 4

[Michael Wu [:mwu]]

(In reply to comment #3)
> Michael, before I submit this to review again, you seem to be the one who added
> the
>    sqlite3Os DATA
> line to sqlite.def originally. Do you think moving the DATA around like in this
> patch will work or do you have another idea?
> 
It does seem to work, but ssl.def (http://lxr.mozilla.org/mozilla/source/security/nss/lib/ssl/ssl.def) didn't need to do this, which is where I looked at originally to add the sqlite3Os DATA line. Don't have any other ideas though..

Comment 5

[Peter Weilbacher]

Do you perhaps know an (online) documentation of the parameters in a Windows .def file? Just to confirm that this still does what it is supposed to do?

Comment 6

[Peter Weilbacher]

Attachment: [checked in] Simpler way

Actually, I found an easier way to do everything with one sed command which also strips off the DATA part without the need to move that around.

Comment 7

[Walter Meinl]

(In reply to comment #5)
> Do you perhaps know an (online) documentation of the parameters in a Windows
> .def file? Just to confirm that this still does what it is supposed to do?
> 

http://support.microsoft.com/kb/132044/
Peter, I'm really no expert but your solution now seems to be more appropriate as the DATA behind sqlite30s has sth to do with the __declspec(dllimport) declared in http://mxr.mozilla.org/seamonkey/source/db/sqlite3/src/sqlite3file.h
I built with your last patch on windows but saw an output that the standalone DATA statement would be ignored (probably with the new patch obsolete to post the concrete warning) though the build went fine and started.
Next question, do we want also declspec(dllimport)

Comment 8

[Peter Weilbacher]

As always I'm a bit confused about the __declspec stuff but if I understand the docs correctly (like item 2 on http://support.microsoft.com/kb/90530/) should sqlite3Os in sqlite3file.h not be decorated with __declspec(dllexport) instead of dllimport? Then it would not need to be listed in the .def file at all.

Comment 9

[Benjamin Smedberg]

Comment on attachment 274799 [details] [diff] [review]
[checked in] Simpler way

This has got to be one of the grosser hacks in our build system.

Comment 10

[Peter Weilbacher]

Comment on attachment 274799 [details] [diff] [review]
[checked in] Simpler way

Checked into trunk.

Michael Wu (or somebody else), any remarks on comment 7/comment 8 or should I close this bug?

Comment 11

[Michael Wu [:mwu]]

(In reply to comment #10)
> (From update of attachment 274799 [details] [diff] [review])
> Checked into trunk.
> 
> Michael Wu (or somebody else), any remarks on comment 7/comment 8 or should I
> close this bug?
> 
No idea (WRT comment 7/comment 8). I just copied what ssl did to make sqlite work..

Comment 12

[Peter Weilbacher]

OK then. What we have now works, let's leave it at that.