Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[oatpp-sqlite] port added #14107

Merged
merged 22 commits into from
Oct 30, 2020

Conversation

KaungZawHtet
Copy link
Contributor

Other oatpp modules hav already been included and this sqlite one left and now here it is

@KaungZawHtet
Copy link
Contributor Author

After seeing the build failure on UWP , I reported to the main author and he replied. Here is the link:
oatpp/oatpp-sqlite#3 (comment)
As you see, build fail at him too. Please investigate the problem at vcpkg.

@NancyLi1013 NancyLi1013 added category:new-port The issue is requesting a new library to be added; consider making a PR! category:port-update The issue is with a library, which is requesting update new revision labels Oct 20, 2020
@NancyLi1013
Copy link
Contributor

The failures on x64-uwp and arm-uwp:

5>D:\buildtrees\oatpp-sqlite\src\d2e5962be1-146fcc8dbb.clean\src\sqlite\sqlite3.c(42196,56): error C2065: 'AreFileApisANSI': undeclared identifier [D:\buildtrees\oatpp-sqlite\x64-uwp-dbg\src\sqlite.vcxproj]
     5>D:\buildtrees\oatpp-sqlite\src\d2e5962be1-146fcc8dbb.clean\src\sqlite\sqlite3.c(42196,56): warning C4312: 'type cast': conversion from 'int' to 'sqlite3_syscall_ptr' of greater size [D:\buildtrees\oatpp-sqlite\x64-uwp-dbg\src\sqlite.vcxproj]
     5>D:\buildtrees\oatpp-sqlite\src\d2e5962be1-146fcc8dbb.clean\src\sqlite\sqlite3.c(42226,52): error C2065: 'CreateFileA': undeclared identifier [D:\buildtrees\oatpp-sqlite\x64-uwp-dbg\src\sqlite.vcxproj]
     5>D:\buildtrees\oatpp-sqlite\src\d2e5962be1-146fcc8dbb.clean\src\sqlite\sqlite3.c(42226,52): warning C4312: 'type cast': conversion from 'int' to 'sqlite3_syscall_ptr' of greater size [D:\buildtrees\oatpp-sqlite\x64-uwp-dbg\src\sqlite.vcxproj]
     5>D:\buildtrees\oatpp-sqlite\src\d2e5962be1-146fcc8dbb.clean\src\sqlite\sqlite3.c(42235,52): error C2065: 'CreateFileW': undeclared identifier [D:\buildtrees\oatpp-sqlite\x64-uwp-dbg\src\sqlite.vcxproj]
     5>D:\buildtrees\oatpp-sqlite\src\d2e5962be1-146fcc8dbb.clean\src\sqlite\sqlite3.c(42235,52): warning C4312: 'type cast': conversion from 'int' to 'sqlite3_syscall_ptr' of greater size [D:\buildtrees\oatpp-sqlite\x64-uwp-dbg\src\sqlite.vcxproj]
     5>D:\buildtrees\oatpp-sqlite\src\d2e5962be1-146fcc8dbb.clean\src\sqlite\sqlite3.c(42385,52): error C2065: 'GetFileSize': undeclared identifier [D:\buildtrees\oatpp-sqlite\x64-uwp-dbg\src\sqlite.vcxproj]
     5>D:\buildtrees\oatpp-sqlite\src\d2e5962be1-146fcc8dbb.clean\src\sqlite\sqlite3.c(42385,52): warning C4312: 'type cast': conversion from 'int' to 'sqlite3_syscall_ptr' of greater size [D:\buildtrees\oatpp-sqlite\x64-uwp-dbg\src\sqlite.vcxproj]
     5>D:\buildtrees\oatpp-sqlite\src\d2e5962be1-146fcc8dbb.clean\src\sqlite\sqlite3.c(42531,53): error C2065: 'HeapValidate': undeclared identifier [D:\buildtrees\oatpp-sqlite\x64-uwp-dbg\src\sqlite.vcxproj]
     5>D:\buildtrees\oatpp-sqlite\src\d2e5962be1-146fcc8dbb.clean\src\sqlite\sqlite3.c(42531,53): warning C4312: 'type cast': conversion from 'int' to 'sqlite3_syscall_ptr' of greater size [D:\buildtrees\oatpp-sqlite\x64-uwp-dbg\src\sqlite.vcxproj]
     5>D:\buildtrees\oatpp-sqlite\src\d2e5962be1-146fcc8dbb.clean\src\sqlite\sqlite3.c(42548,53): error C2065: 'LoadLibraryA': undeclared identifier [D:\buildtrees\oatpp-sqlite\x64-uwp-dbg\src\sqlite.vcxproj]
     5>D:\buildtrees\oatpp-sqlite\src\d2e5962be1-146fcc8dbb.clean\src\sqlite\sqlite3.c(42548,53): warning C4312: 'type cast': conversion from 'int' to 'sqlite3_syscall_ptr' of greater size [D:\buildtrees\oatpp-sqlite\x64-uwp-dbg\src\sqlite.vcxproj]
     5>D:\buildtrees\oatpp-sqlite\src\d2e5962be1-146fcc8dbb.clean\src\sqlite\sqlite3.c(42557,53): error C2065: 'LoadLibraryW': undeclared identifier 

The error occurred when building sqlite3.c file.

Since sqlite3 has been provided in vcpkg now.
We should depend on sqlite3 in vcpkg to build oatpp-sqlite instead of building the source codes embedded in oatpp-sqlite here https://github.com/oatpp/oatpp-sqlite/tree/master/src/sqlite.

So could you please update this to rebuild this again?

ports/oatpp-sqlite/portfile.cmake Outdated Show resolved Hide resolved
ports/oatpp-sqlite/portfile.cmake Outdated Show resolved Hide resolved
@KaungZawHtet
Copy link
Contributor Author

fixed but failed. For the failure, is there any informative log that need to shared here ?

@NancyLi1013
Copy link
Contributor

CMake Error at D:/downloads/tools/cmake-3.18.4-windows/cmake-3.18.4-win32-x86/share/cmake-3.18/Modules/FindPackageHandleStandardArgs.cmake:165 (message):
  Could NOT find SQLite3 (missing: SQLite3_INCLUDE_DIR SQLite3_LIBRARY)
Call Stack (most recent call first):
  D:/downloads/tools/cmake-3.18.4-windows/cmake-3.18.4-win32-x86/share/cmake-3.18/Modules/FindPackageHandleStandardArgs.cmake:458 (_FPHSA_FAILURE_MESSAGE)
  D:/downloads/tools/cmake-3.18.4-windows/cmake-3.18.4-win32-x86/share/cmake-3.18/Modules/FindSQLite3.cmake:52 (find_package_handle_standard_args)
  C:/a/1/s/scripts/buildsystems/vcpkg.cmake:536 (_find_package)
  CMakeLists.txt:108 (find_package)

Please check the log here
https://dev.azure.com/vcpkg/public/_build/results?buildId=44522&view=artifacts&type=publishedArtifacts

@KaungZawHtet
Copy link
Contributor Author

Fixed and builds are in success now

Copy link
Contributor

@NancyLi1013 NancyLi1013 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Could you please also bump CONTROL file?

Add Port-Version for all ports except for new port oatpp-sqlite.

ports/oatpp-consul/portfile.cmake Outdated Show resolved Hide resolved
ports/oatpp-consul/portfile.cmake Outdated Show resolved Hide resolved
ports/oatpp-libressl/portfile.cmake Outdated Show resolved Hide resolved
ports/oatpp-libressl/portfile.cmake Outdated Show resolved Hide resolved
ports/oatpp-mbedtls/portfile.cmake Outdated Show resolved Hide resolved
ports/oatpp-websocket/portfile.cmake Outdated Show resolved Hide resolved
ports/oatpp/portfile.cmake Outdated Show resolved Hide resolved
ports/oatpp/portfile.cmake Outdated Show resolved Hide resolved
ports/oatpp/portfile.cmake Outdated Show resolved Hide resolved
ports/oatpp/portfile.cmake Show resolved Hide resolved
@KaungZawHtet
Copy link
Contributor Author

1.2.0 is only version number main author gives us. No bump version we get to display

@NancyLi1013
Copy link
Contributor

1.2.0 is only version number main author gives us. No bump version we get to display

You're right, the version number is from upstream. But we also need to bump CONTROL file. In this case, we can add Port-Version: 1. Since there are changes in these ports.

Please see this doc https://github.com/microsoft/vcpkg/blob/master/docs/maintainers/control-files.md#port-version

@KaungZawHtet
Copy link
Contributor Author

Port-Version: 2 to oatpp and other oatpp modules except oatpp-sqlite , right ?

ports/oatpp/CONTROL Outdated Show resolved Hide resolved
@NancyLi1013
Copy link
Contributor

Port-Version: 2 to oatpp and other oatpp modules except oatpp-sqlite , right ?

All Port-Version: should be 1, except oatpp-sqlite.

@NancyLi1013 NancyLi1013 added the info:reviewed Pull Request changes follow basic guidelines label Oct 29, 2020
@BillyONeal BillyONeal merged commit d89bfc3 into microsoft:master Oct 30, 2020
@BillyONeal
Copy link
Member

Thanks again!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
category:new-port The issue is requesting a new library to be added; consider making a PR! category:port-update The issue is with a library, which is requesting update new revision info:reviewed Pull Request changes follow basic guidelines
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

4 participants