xpra icon
Bug tracker and wiki

Opened 3 years ago

Closed 3 years ago

#916 closed enhancement (fixed)

tidy up the win32 build

Reported by: Antoine Martin Owned by: Smo
Priority: major Milestone: 0.16
Component: platforms Version: 0.15.x
Keywords: win32 64-bit Cc:

Description

Related to #678 and #640.

  • building on 64-bit windows system is painful, r9920 fixes most of that
  • we have too many inconsistent paths dumping in to C:\

Change History (4)

comment:1 Changed 3 years ago by Antoine Martin

Owner: changed from Antoine Martin to Antoine Martin
Status: newassigned

As of r9921, we can now move all the build libs into a prefix, it still defaults to C:\ for backwards compatibility, but I've moved my test build system to use E:\Xpra-Build-Libs which now looks like this:

ffmpeg2-win32-bin
libwebp-windows-x86
Microsoft.VC90.CRT
Microsoft.VC90.MFC
vpx-1.4
x264

This could also be used to lock a specific version to a specific set of libraries.

Note: we now support both 32-bit and 64-bit helpers (ghoscript for printing, etc) - we still prefer the 32-bit version if present so that we can build 32-bit installers on a 64-bit system which may have both versions of the tools. This has negligible cost: printing is not performance critical, but may have security implications as 64-bit tends to be safer. (this could be changed later)

Last edited 3 years ago by Antoine Martin (previous) (diff)

comment:2 Changed 3 years ago by Antoine Martin

For the record, this is how I built stuff with msvc 2008:

Then copy the headers and lib files to the python installation directories.

Not done yet:

Last edited 3 years ago by Antoine Martin (previous) (diff)

comment:3 Changed 3 years ago by Antoine Martin

Owner: changed from Antoine Martin to Smo
Status: assignednew

r10052 backported this to v0.14.x and v0.15.x, and we can now have a different lib directory for each branch.
ie: v0.15.x will try to find:

  • E:\Xpra-Build-Libs-v0.15.x\
  • E:\Xpra-Build-Libs\
  • C:\

So everything should work as before for all branches, and you can either use a common directory for all, or individual directories as needed.

@smo: please close to ACK, assuming nothing is broken...

comment:4 Changed 3 years ago by Smo

Resolution: fixed
Status: newclosed

Tested this works as expected. I will probably leave mine at the default location for now but I like how this works now.

Note: See TracTickets for help on using tickets.