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

vg does not compile on Arch Linux #2522

Closed
subwaystation opened this issue Oct 31, 2019 · 3 comments · Fixed by #2523
Closed

vg does not compile on Arch Linux #2522

subwaystation opened this issue Oct 31, 2019 · 3 comments · Fixed by #2523

Comments

@subwaystation
Copy link
Member

subwaystation commented Oct 31, 2019

1. What were you trying to do?
Compile vg on Arch Linux.

git clone --recursive https://github.com/vgteam/vg.git
cd vg
. ./source_me.sh && make

2. What did you want to happen?
It did not compile.

3. What actually happened?
Make crashed.

4. If you got a line like Stack trace path: /somewhere/on/your/computer/stacktrace.txt, please copy-paste the contents of that file here:

. ./source_me.sh && /usr/bin/g++ -I/home/heumos/git/vg/include -I. -I/home/heumos/git/vg/src -I/home/heumos/git/vg/src/unittest -I/home/heumos/git/vg/src/subcommand -I/home/heumos/git/vg/include/dynamic -I/home/heumos/git/vg/include/sonLib -I/usr/include/cairo -I/usr/include/glib-2.0 -I/usr/lib/glib-2.0/include -I/usr/lib/libffi-3.2.1/include -I/usr/include/pixman-1 -I/usr/include/freetype2 -I/usr/include/libpng16 -I/usr/include/harfbuzz  -O3 -Werror=return-type -std=c++14 -ggdb -g -MMD -MP -I /home/heumos/git/vg/include -I/home/heumos/git/vg/include/dynamic -I /home/heumos/git/vg/include -I/home/heumos/git/vg/include/dynamic  -fopenmp -msse4.2 -c -o obj/main.o src/main.cpp 
. ./source_me.sh && /usr/bin/g++ -I/home/heumos/git/vg/include -I. -I/home/heumos/git/vg/src -I/home/heumos/git/vg/src/unittest -I/home/heumos/git/vg/src/subcommand -I/home/heumos/git/vg/include/dynamic -I/home/heumos/git/vg/include/sonLib -I/usr/include/cairo -I/usr/include/glib-2.0 -I/usr/lib/glib-2.0/include -I/usr/lib/libffi-3.2.1/include -I/usr/include/pixman-1 -I/usr/include/freetype2 -I/usr/include/libpng16 -I/usr/include/harfbuzz  -O3 -Werror=return-type -std=c++14 -ggdb -g -MMD -MP -I /home/heumos/git/vg/include -I/home/heumos/git/vg/include/dynamic -I /home/heumos/git/vg/include -I/home/heumos/git/vg/include/dynamic  -fopenmp -msse4.2 -c -o obj/haplotype_extracter.o src/haplotype_extracter.cpp 
. ./source_me.sh && /usr/bin/g++ -I/home/heumos/git/vg/include -I. -I/home/heumos/git/vg/src -I/home/heumos/git/vg/src/unittest -I/home/heumos/git/vg/src/subcommand -I/home/heumos/git/vg/include/dynamic -I/home/heumos/git/vg/include/sonLib -I/usr/include/cairo -I/usr/include/glib-2.0 -I/usr/lib/glib-2.0/include -I/usr/lib/libffi-3.2.1/include -I/usr/include/pixman-1 -I/usr/include/freetype2 -I/usr/include/libpng16 -I/usr/include/harfbuzz  -O3 -Werror=return-type -std=c++14 -ggdb -g -MMD -MP -I /home/heumos/git/vg/include -I/home/heumos/git/vg/include/dynamic -I /home/heumos/git/vg/include -I/home/heumos/git/vg/include/dynamic  -fopenmp -msse4.2 -c -o obj/multipath_mapper.o src/multipath_mapper.cpp 
. ./source_me.sh && /usr/bin/g++ -I/home/heumos/git/vg/include -I. -I/home/heumos/git/vg/src -I/home/heumos/git/vg/src/unittest -I/home/heumos/git/vg/src/subcommand -I/home/heumos/git/vg/include/dynamic -I/home/heumos/git/vg/include/sonLib -I/usr/include/cairo -I/usr/include/glib-2.0 -I/usr/lib/glib-2.0/include -I/usr/lib/libffi-3.2.1/include -I/usr/include/pixman-1 -I/usr/include/freetype2 -I/usr/include/libpng16 -I/usr/include/harfbuzz  -O3 -Werror=return-type -std=c++14 -ggdb -g -MMD -MP -I /home/heumos/git/vg/include -I/home/heumos/git/vg/include/dynamic -I /home/heumos/git/vg/include -I/home/heumos/git/vg/include/dynamic  -fopenmp -msse4.2 -c -o obj/alignment_emitter.o src/alignment_emitter.cpp 
. ./source_me.sh && /usr/bin/g++ -I/home/heumos/git/vg/include -I. -I/home/heumos/git/vg/src -I/home/heumos/git/vg/src/unittest -I/home/heumos/git/vg/src/subcommand -I/home/heumos/git/vg/include/dynamic -I/home/heumos/git/vg/include/sonLib -I/usr/include/cairo -I/usr/include/glib-2.0 -I/usr/lib/glib-2.0/include -I/usr/lib/libffi-3.2.1/include -I/usr/include/pixman-1 -I/usr/include/freetype2 -I/usr/include/libpng16 -I/usr/include/harfbuzz  -O3 -Werror=return-type -std=c++14 -ggdb -g -MMD -MP -I /home/heumos/git/vg/include -I/home/heumos/git/vg/include/dynamic -I /home/heumos/git/vg/include -I/home/heumos/git/vg/include/dynamic  -fopenmp -msse4.2 -c -o obj/chunker.o src/chunker.cpp 
. ./source_me.sh && /usr/bin/g++ -I/home/heumos/git/vg/include -I. -I/home/heumos/git/vg/src -I/home/heumos/git/vg/src/unittest -I/home/heumos/git/vg/src/subcommand -I/home/heumos/git/vg/include/dynamic -I/home/heumos/git/vg/include/sonLib -I/usr/include/cairo -I/usr/include/glib-2.0 -I/usr/lib/glib-2.0/include -I/usr/lib/libffi-3.2.1/include -I/usr/include/pixman-1 -I/usr/include/freetype2 -I/usr/include/libpng16 -I/usr/include/harfbuzz  -O3 -Werror=return-type -std=c++14 -ggdb -g -MMD -MP -I /home/heumos/git/vg/include -I/home/heumos/git/vg/include/dynamic -I /home/heumos/git/vg/include -I/home/heumos/git/vg/include/dynamic  -fopenmp -msse4.2 -c -o obj/stream_sorter.o src/stream_sorter.cpp 
. ./source_me.sh && /usr/bin/g++ -I/home/heumos/git/vg/include -I. -I/home/heumos/git/vg/src -I/home/heumos/git/vg/src/unittest -I/home/heumos/git/vg/src/subcommand -I/home/heumos/git/vg/include/dynamic -I/home/heumos/git/vg/include/sonLib -I/usr/include/cairo -I/usr/include/glib-2.0 -I/usr/lib/glib-2.0/include -I/usr/lib/libffi-3.2.1/include -I/usr/include/pixman-1 -I/usr/include/freetype2 -I/usr/include/libpng16 -I/usr/include/harfbuzz  -O3 -Werror=return-type -std=c++14 -ggdb -g -MMD -MP -I /home/heumos/git/vg/include -I/home/heumos/git/vg/include/dynamic -I /home/heumos/git/vg/include -I/home/heumos/git/vg/include/dynamic  -fopenmp -msse4.2 -c -o obj/variant_recall.o src/variant_recall.cpp 
In file included from src/vg.hpp:29,
                 from src/variant_recall.hpp:16,
                 from src/variant_recall.cpp:2:
/home/heumos/git/vg/include/vg/io/stream.hpp: In function ‘void vg::io::for_each_parallel_impl(std::istream&, const std::function<void(T&, T&)>&, const std::function<void(T&)>&, const std::function<bool()>&, size_t) [with T = vg::Alignment]’:
/home/heumos/git/vg/include/vg/io/stream.hpp:185:34: error: ‘batch_size’ not specified in enclosing ‘parallel’
  185 |             if (batch->size() == batch_size) {
      |                                  ^~~~~~~~~~
/home/heumos/git/vg/include/vg/io/stream.hpp:153:13: error: enclosing ‘parallel’
  153 |     #pragma omp parallel default(none) shared(in, lambda1, lambda2, batches_outstanding, max_batches_outstanding, single_threaded_until_true, cerr)
      |             ^~~
/home/heumos/git/vg/include/vg/io/stream.hpp:242:47: error: ‘batch_size’ not specified in enclosing ‘task’
  242 |                             for (int i = 0; i<batch_size; i+=2) {
      |                                               ^~~~~~~~~~
/home/heumos/git/vg/include/vg/io/stream.hpp:234:9: error: enclosing ‘task’
  234 | #pragma omp task default(none) firstprivate(batch) shared(batches_outstanding, lambda2, handle, single_threaded_until_true, cerr)
      |         ^~~
make: *** [Makefile:693: obj/variant_recall.o] Error 1

5. What data and command can the vg dev team use to make the problem happen?
See above.

6. What does running vg version say?

Current master clone from github.

Could someone please help me out here?

@subwaystation
Copy link
Member Author

[heumos@wave vg]$ gcc --version
gcc (GCC) 9.2.0

@glennhickey
Copy link
Contributor

Hmm, I guess this is my fault. On my machine (and the CI systems), "batch_size" get read even with #pragma omp parallel default(none) because it is const, which OMP implictly shares. I guess newer compilers (I'm using 7.4) don't take kindly to this.

@subwaystation
Copy link
Member Author

Thanks man, that solved it :)

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

Successfully merging a pull request may close this issue.

2 participants