Opened 8 years ago
Closed 8 years ago
#6815 closed enhancement (fixed)
Update makgdal_gen.bat to generate project for autotest/cpp
Reported by: | Mateusz Łoskot | Owned by: | Mateusz Łoskot |
---|---|---|---|
Priority: | normal | Milestone: | |
Component: | ConfigBuild | Version: | svn-trunk |
Severity: | normal | Keywords: | |
Cc: |
Description
This patch updates makgdal_gen.bat to:
- Generate separate
.vcxproj
file for GDAL C++ tests- The new project is created only if autotest/cpp folder exists (ie. source code tree downloaded from SVN/Git repo)
- Add
CPL_DEBUG
andGDAL_DATA
variables toLocalDebuggerEnvironment
in.user
file. - Use
setlocal
to keep the command prompt environment clean. - Update list of subfolders excluded from the main project.
- Clean up / clarify usage instructions.
I'll appreciate review of my changes.
Next, I'd like to propose renaming of the script from makegdal_gen.bat
to generate_vcxproj.bat
, so the name reflects the purpose.
If we agree to rename it, I will update the Wiki pages where the old name occurs.
Change History (10)
comment:1 by , 8 years ago
comment:2 by , 8 years ago
Cc: | removed |
---|---|
Owner: | changed from | to
comment:3 by , 8 years ago
Due to problems with attaching the patch I decided to submit the changes via GitHub PR: https://github.com/OSGeo/gdal/pull/195
comment:4 by , 8 years ago
I haven't tried but I trust it will work fine since Mateusz is the primary user of that. The name change is also a good idea.
comment:5 by , 8 years ago
@ilucena Yes, I use it myself and it works. I will merge into trunk. Thanks.
I will also rename.
I also have an idea to update the script so it generates a basic .sln file with the two .vcxproj attached. I may add it later, separate commit.
comment:8 by , 8 years ago
Wiki pages updated.
The current page is wiki:GeneratingVisualStudioProject
comment:9 by , 8 years ago
Owner: | changed from | to
---|
comment:10 by , 8 years ago
Resolution: | → fixed |
---|---|
Status: | new → closed |
Ups, I'm not being allowed to submit any files. Reported to admins: https://trac.osgeo.org/osgeo/ticket/1877