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

sqlite-orm build failure #21996

Closed
quickiwiki opened this issue Dec 12, 2021 · 11 comments · Fixed by #22253
Closed

sqlite-orm build failure #21996

quickiwiki opened this issue Dec 12, 2021 · 11 comments · Fixed by #22253
Assignees
Labels
category:port-bug The issue is with a library, which is something the port should already support

Comments

@quickiwiki
Copy link
Contributor

quickiwiki commented Dec 12, 2021

Host Environment

  • OS: Windows/Linux
  • Compiler: VS 2017/g++ (Ubuntu 8.4.0-1ubuntu1~18.04) 8.4.0

To Reproduce
Steps to reproduce the behavior:
./vcpkg install sqlite-orm

Failure logs

/vcpkg/installed/x64-linux/include/sqlite_orm/sqlite_orm.h:10124:25: error: ‘sqlite_orm::internal::is_upsert_clause’ has not been declared
         using internal::is_upsert_clause;
                         ^~~~~~~~~~~~~~~~
/vcpkg/installed/x64-linux/include/sqlite_orm/sqlite_orm.h:10146:68: error: ‘is_upsert_clause’ was not declared in this scope
         constexpr int upsertClausesCount = count_tuple<args_tuple, is_upsert_clause>::value;
                                                                    ^~~~~~~~~~~~~~~~

Additional context
The previous version of the lib (e.g.: sqlite-orm:x64-linux 1.6) worked correctly

@JackBoosY JackBoosY added the requires:repro The issue is not currently repro-able label Dec 13, 2021
@JackBoosY
Copy link
Contributor

Can you please provide the following logs:
buildtrees/sqlite-orm/config-x64-linux-dbg-out.log
buildtrees/sqlite-orm/install-x64-linux-dbg-out.log

Thanks.

@JackBoosY JackBoosY added the requires:more-information This Issue requires more information to solve label Dec 13, 2021
@quickiwiki
Copy link
Contributor Author

quickiwiki commented Dec 13, 2021

Thanks, this happens when I try to use sqlite-orm in a project. I may have submitted to the wrong category

cat ~/vcpkg/buildtrees/sqlite-orm/config-x64-linux-dbg-out.log
-- The C compiler identification is GNU 8.4.0
-- The CXX compiler identification is GNU 8.4.0
-- Detecting C compiler ABI info
-- Detecting C compiler ABI info - done
-- Check for working C compiler: /usr/bin/cc - skipped
-- Detecting C compile features
-- Detecting C compile features - done
-- Detecting CXX compiler ABI info
-- Detecting CXX compiler ABI info - done
-- Check for working CXX compiler: /usr/bin/c++ - skipped
-- Detecting CXX compile features
-- Detecting CXX compile features - done
-- SQLITE_ORM: Build with C++14 features
-- Configuring sqlite_orm 1.6.0
-- Looking for pthread.h
-- Looking for pthread.h - found
-- Performing Test CMAKE_HAVE_LIBC_PTHREAD
-- Performing Test CMAKE_HAVE_LIBC_PTHREAD - Failed
-- Looking for pthread_create in pthreads
-- Looking for pthread_create in pthreads - not found
-- Looking for pthread_create in pthread
-- Looking for pthread_create in pthread - found
-- Found Threads: TRUE
-- CMAKE_BUILD_TYPE: Debug
-- Configuring done
-- Generating done
-- Build files have been written to: /home/el/vcpkg/buildtrees/sqlite-orm/x64-linux-dbg

cat ~/vcpkg/buildtrees/sqlite-orm/install-x64-linux-dbg-out.log

[0/1] cd /home/el/vcpkg/buildtrees/sqlite-orm/x64-linux-dbg && /home/el/vcpkg/downloads/tools/cmake-3.21.1-linux/cmake-3.21.1-linux-x86_64/bin/cmake -P cmake_install.cmake
-- Install configuration: "Debug"
-- Installing: /home/el/vcpkg/packages/sqlite-orm_x64-linux/debug/lib/cmake/SqliteOrm/SqliteOrmConfig.cmake
-- Installing: /home/el/vcpkg/packages/sqlite-orm_x64-linux/debug/lib/cmake/SqliteOrm/SqliteOrmConfigVersion.cmake
-- Installing: /home/el/vcpkg/packages/sqlite-orm_x64-linux/debug/include/sqlite_orm
-- Installing: /home/el/vcpkg/packages/sqlite-orm_x64-linux/debug/include/sqlite_orm/sqlite_orm.h
-- Installing: /home/el/vcpkg/packages/sqlite-orm_x64-linux/debug/lib/cmake/SqliteOrm/SqliteOrmTargets.cmake

@JackBoosY JackBoosY removed the requires:more-information This Issue requires more information to solve label Dec 13, 2021
@JackBoosY
Copy link
Contributor

Okay, can you also please provide a sample code to reproduce this?

Thanks.

@JackBoosY
Copy link
Contributor

Ping @quickiwiki for response.

@JackBoosY JackBoosY added the requires:more-information This Issue requires more information to solve label Dec 15, 2021
@quickiwiki
Copy link
Contributor Author

Yes, sorry @JackBoosY I'll provide soon the sample project

@quickiwiki
Copy link
Contributor Author

@JackBoosY
Copy link
Contributor

@quickiwiki The repo you provided is not exists anymore, can you please provide it again?

Thanks.

@quickiwiki
Copy link
Contributor Author

@JackBoosY Sorry, could you please try now?

@JackBoosY
Copy link
Contributor

1>f:\vcpkg\installed\x86-windows\include\sqlite_orm\sqlite_orm.h(10146): fatal error C1001: An internal error has occurred in the compiler.

@JackBoosY
Copy link
Contributor

JackBoosY commented Dec 21, 2021

Confirmed.
This issue seems like a upstream bug and was fixed in the master branch.
I will update this port to resolve it.

More details see fnc12/sqlite_orm#841.

@JackBoosY JackBoosY added category:port-bug The issue is with a library, which is something the port should already support category:port-update The issue is with a library, which is requesting update new revision and removed requires:repro The issue is not currently repro-able requires:more-information This Issue requires more information to solve labels Dec 21, 2021
@JackBoosY
Copy link
Contributor

No, this issue is not fixed in the latest master (commit ID: 115c6f64a340eac5bc5509bf884a7f350c9e6aa8).
Can you please report this to the upstream?

Thanks.

@JackBoosY JackBoosY added depends:upstream-changes Waiting on a change to the upstream project and removed category:port-update The issue is with a library, which is requesting update new revision labels Dec 21, 2021
trueqbit pushed a commit to FireDaemon/vcpkg that referenced this issue Dec 29, 2021
BillyONeal pushed a commit that referenced this issue Dec 30, 2021
@JackBoosY JackBoosY removed the depends:upstream-changes Waiting on a change to the upstream project label Dec 30, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
category:port-bug The issue is with a library, which is something the port should already support
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants