OpenLayers OpenLayers

Ticket #385 (closed feature: fixed)

Opened 2 years ago

Last modified 2 years ago

building for windows users

Reported by: tschaub Assigned to: sderle
Priority: minor Milestone: 2.3 Release
Component: general Version: 2.3 RC1
Keywords: Cc:
State:

Description

Think this ticket is captured elsewhere, but I couldn't identify exactly where. I know the real solution is to have a pure python build script, but to satisfy any windows users in the meantime, I've put together a quick batch script. This requires one simple change to tools/mergejs.py.

The mergejs.py change might make sense to incorporate anyway. I didn't take time to track it down, but it seems that toposort produces some / delimited paths. In any case, everything works if you treat all paths as / delimited - including in the cfg file.

Since I've been a little patch happy lately, I thought I'd hold off. If anybody is interested in the changes, they're in r1747.

In addition to the one line addition in mergejs.py, I've replaced a few tabs with spaces.

No guarantees, but works for me.

Attachments

license_build.patch (0.6 kB) - added by tschaub on 01/03/07 18:52:41.
patch to build with license

Change History

11/16/06 16:52:19 changed by sderle

  • owner set to sderle.
  • status changed from new to assigned.

11/16/06 19:19:04 changed by tschaub

In case the above is too cryptic, Windows users can build with this batch script: http://trac.openlayers.org/browser/sandbox/tschaub/wc/build/build.bat

And the one line addition to mergejs.py shown in r1747.

Until you've got the pure python build worked out, this might be a suitable alternative.

12/06/06 16:05:28 changed by euzuro

  • milestone set to 2.3 Release.

12/22/06 15:14:34 changed by sderle

  • status changed from assigned to closed.
  • resolution set to fixed.

Fixed by r2097 and r2098.

01/03/07 18:52:17 changed by tschaub

  • status changed from closed to reopened.
  • resolution deleted.

jsmin strips out the license file - patch to follow

01/03/07 18:52:41 changed by tschaub

  • attachment license_build.patch added.

patch to build with license

01/03/07 18:53:56 changed by tschaub

  • keywords set to review.

happy to commit if reviewed - note this applies to all users, not just windows users

01/03/07 20:05:25 changed by euzuro

  • keywords deleted.
  • version set to 2.3 RC1.

nice one tim :-)

I'm marking the version on this 2.3 RC1

Please go ahead and apply this patch in trunk, mark ticket as pullup.

01/04/07 16:31:24 changed by tschaub

  • keywords set to pullup.

applied in r2126

01/22/07 22:14:21 changed by crschmidt

  • keywords deleted.
  • status changed from reopened to closed.
  • resolution set to fixed.