Skip to content
This repository has been archived by the owner on Oct 10, 2023. It is now read-only.

Setup dev build so only one list of source files is need #88

Closed
benbu opened this issue Aug 8, 2013 · 1 comment
Closed

Setup dev build so only one list of source files is need #88

benbu opened this issue Aug 8, 2013 · 1 comment
Assignees
@benbu
Labels
enhancement

Comments

@benbu
Copy link
Contributor

benbu commented Aug 8, 2013

Also make JS_LINT_FILES just use JS_UGLIFY_FILES
@ghost ghost assigned benbu Aug 8, 2013
@benbu
Copy link
Contributor Author

benbu commented Aug 8, 2013

Added in branch 88-enhance-run-from-source-config

This makes it so you only need to add new files to a single list
in the CMakeLists.txt file. The source based geoweb.min.js is now
autogenerated using the JS_UGLIFY_FILES list. Additionally,
JS_LINT_FILES originates from JS_UGLIFY_FILES now, leaving a single
list of source files in CMakeLists.txt

@cjh1 can you review this and get it into master?

benbu pushed a commit that referenced this issue Aug 8, 2013
Combines source file lists into one #88
cd03c8a 
This makes it so you only need to add new files to a single list
in the CMakeLists.txt file. The source based geoweb.min.js is now
autogenerated using the JS_UGLIFY_FILES list. Additionally,
JS_LINT_FILES originates from JS_UGLIFY_FILES now, leaving a single
list of source files in CMakeLists.txt
aashish24 added a commit that referenced this issue Aug 12, 2013
@aashish24
Merge remote-tracking branch 'origin/88-enhance-run-from-source-config'
5ea789e 
* origin/88-enhance-run-from-source-config:
  Adds missing file
  ignore src geoweb.min.js since its auto-generated
  Combines source file lists into one #88

Conflicts:
	web/lib/geoweb.min.js
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Assignees

@benbu benbu

Labels
enhancement
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
2 participants
@aashish24 @benbu