jmcnamara / MSVCLibXlsxWriter

A MSVC project to build a Windows DLL for libxlsxwriter
http://libxlsxwriter.github.io
22 stars 15 forks source link

Doesn't compile fine for any configuration #2

Closed hlide closed 7 years ago

hlide commented 7 years ago

I selected "Release/x86" configuration then click "Build", I got compiling errors.

It is clear that those visual studio project files were never tested against all configurations (at least it wasn't for win32).

Github is plagued with such half-working projects and so less compelling to embrace.

jmcnamara commented 7 years ago

Github is plagued with such half-working projects

GitHub is also plagued with incomplete bug reports.

Regards,

John.

hlide commented 7 years ago

It only worked with "Debug/x64" configuration. I had to modify project stuff like "include path" to make it work with the other configurations.

For your record, I also tried with cmake on the other repository "libxlsxwriter" but failed because it was unable to retrieve stuff like zlib. I followed what was written on "https://libxlsxwriter.github.io/getting_started.html#gsg_cmake_generators". However, I did try cmake only on release 0.7.0.

hlide commented 7 years ago

Let me recheck something...

hlide commented 7 years ago

So, I retried the following case:

I got errors such: 1> app.c 1>..\..\libxlsxwriter\src\app.c(10): fatal error C1083: Cannot open include file: 'xlsxwriter/xmlwriter.h': No such file or directory. If you look at project property, you can see the other configurations have a faulty include path compared with the default configuration. So this is a half-working project file.

jmcnamara commented 7 years ago

I retried the following case: clone with GIT this repository

What command did you use?

hlide commented 7 years ago

git.exe clone -v --recurse-submodules --progress "https://github.com/jmcnamara/MSVCLibXlsxWriter.git" "C:/GIT/MSVCLibXlsxWriter3"

hlide commented 7 years ago

here the output of the command:

Cloning into 'C:/GIT/MSVCLibXlsxWriter3'...                                                  
POST git-upload-pack (175 bytes)                                                             
remote: Counting objects: 124, done.                                                         
Rremote: Total 124 (delta 0), reused 0 (delta 0), pack-reused 124                            
Receiving objects:  49% (61/124), 956.01 KiB | 1.82 MiB/s                                    
Receiving objects: 100% (124/124), 1.45 MiB | 1.82 MiB/s, done.                              
Resolving deltas: 100% (61/61), done.                                                        
Submodule 'libxlsxwriter' (https://github.com/jmcnamara/libxlsxwriter.git) registered for path 'libxlsxwriter'                                                                            
Cloning into 'C:/GIT/MSVCLibXlsxWriter3/libxlsxwriter'...                                    
remote: Counting objects: 7591, done.                                                        
remote: Compressing objects: 100% (11/11), done.                                             
remote: Total 7591 (delta 2), reused 10 (delta 2), pack-reused 7578                          
Receiving objects: 100% (7591/7591), 13.70 MiB | 6.15 MiB/s, done.                           
Resolving deltas: 100% (5622/5622), done.                                                    
Submodule path 'libxlsxwriter': checked out '56064038a40a6a5393b71e03e48ca369b964ae4f'       
Done
jmcnamara commented 7 years ago

Fixed on master.