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

Build error on 'main' branch #1765

Closed
ivan-tkatchev opened this issue Jun 11, 2024 · 3 comments
Closed

Build error on 'main' branch #1765

ivan-tkatchev opened this issue Jun 11, 2024 · 3 comments

Comments

@ivan-tkatchev
Copy link

What happens?

[ 95%] Building CXX object CMakeFiles/duckdb_web.dir/src/webdb_api.cc.o
/build/source/lib/src/io/web_filesystem.cc:648:78: error: 'flags' is a private member of 'duckdb::FileOpenFlags'
  648 |                 auto *opened = duckdb_web_fs_file_open(file->file_id_, flags.flags);
      |                                                                              ^
/build/source/build/relperf/mvp/third_party/duckdb/install/include/duckdb/common/file_open_flags.hpp:104:8: note: declared private here
  104 |         idx_t flags = 0;
      |               ^
1 error generated.

To Reproduce

Any attempt to build will fail, see error message.

Browser/Environment:

none

Device:

none

DuckDB-Wasm Version:

main

DuckDB-Wasm Deployment:

none

Full Name:

Ivan Tkachev

Affiliation:

Hyper AdTech

@carlopi
Copy link
Collaborator

carlopi commented Jun 11, 2024

Hi! This I need to fix at the Makefile level, at the moment you need to do something like:

patch -p1 -d submodules/duckdb < cat duckdb.patch

or equivalent

cd submodules/duckdb
git apply ../../duckdb.patch
cd ../..

before building.

@attila-kun
Copy link

Before building, I did make patch_duckdb, then it worked.

@carlopi
Copy link
Collaborator

carlopi commented Aug 23, 2024

I think this should be solved now, at the moment you need make apply_patches

@carlopi carlopi closed this as completed Aug 23, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

3 participants