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

Unable to compile project #41

Open
raphaelgault opened this issue Dec 28, 2023 · 2 comments · May be fixed by #43
Open

Unable to compile project #41

raphaelgault opened this issue Dec 28, 2023 · 2 comments · May be fixed by #43

Comments

@raphaelgault
Copy link

raphaelgault commented Dec 28, 2023

Hello,

I just followed the README instructions for building the project but got a compile error:

[ 53%] Building CXX object CMakeFiles/PaxOS.dir/src/lua/lua.cpp.o
In file included from /home/rgault/Documents/PersonnalProjects/PaxOS-8/src/lua/../widgets/gui/../../interface/LovyanGFX/lgfx/v1_init.hpp:25,
                 from /home/rgault/Documents/PersonnalProjects/PaxOS-8/src/lua/../widgets/gui/../../interface/LovyanGFX/LovyanGFX.hpp:31,
                 from /home/rgault/Documents/PersonnalProjects/PaxOS-8/src/lua/../widgets/gui/../../interface/screen.hpp:19,
                 from /home/rgault/Documents/PersonnalProjects/PaxOS-8/src/lua/../widgets/gui/gui_class.hpp:16,
                 from /home/rgault/Documents/PersonnalProjects/PaxOS-8/src/lua/../widgets/gui.hpp:4,
                 from /home/rgault/Documents/PersonnalProjects/PaxOS-8/src/lua/lua2.hpp:3,
                 from /home/rgault/Documents/PersonnalProjects/PaxOS-8/src/lua/lua.hpp:4,
                 from /home/rgault/Documents/PersonnalProjects/PaxOS-8/src/lua/lua.cpp:1:
/home/rgault/Documents/PersonnalProjects/PaxOS-8/src/lua/../widgets/gui/../../interface/LovyanGFX/lgfx/v1/LGFXBase.hpp: In instantiation of ‘void lgfx::v1::LGFXBase::setColor(T) [with T = long long int]’:
/home/rgault/Documents/PersonnalProjects/PaxOS-8/src/lua/../widgets/gui/../../interface/LovyanGFX/lgfx/v1/LGFXBase.hpp:109:123:   required from ‘void lgfx::v1::LGFXBase::fillRect(int32_t, int32_t, int32_t, int32_t, const T&) [with T = long long int; int32_t = int]’
/home/rgault/Documents/PersonnalProjects/PaxOS-8/src/lua/lua.cpp:530:59:   required from here
/home/rgault/Documents/PersonnalProjects/PaxOS-8/src/lua/../widgets/gui/../../interface/LovyanGFX/lgfx/v1/LGFXBase.hpp:76:75: error: call of overloaded ‘convert(long long int&)’ is ambiguous
   76 |     LGFX_INLINE_T void setColor(T color) { setRawColor(_write_conv.convert(color)); }
      |                                                        ~~~~~~~~~~~~~~~~~~~^~~~~~~
In file included from /home/rgault/Documents/PersonnalProjects/PaxOS-8/src/lua/../widgets/gui/../../interface/LovyanGFX/lgfx/v1/LGFXBase.hpp:29:
/home/rgault/Documents/PersonnalProjects/PaxOS-8/src/lua/../widgets/gui/../../interface/LovyanGFX/lgfx/v1/misc/colortype.hpp:850:26: note: candidate: ‘uint32_t lgfx::v1::color_conv_t::convert(const lgfx::v1::rgb332_t&)’
  850 |     LGFX_INLINE uint32_t convert(const rgb332_t&   c) { return convert_rgb332(  c.get()); }
      |                          ^~~~~~~
/home/rgault/Documents/PersonnalProjects/PaxOS-8/src/lua/../widgets/gui/../../interface/LovyanGFX/lgfx/v1/misc/colortype.hpp:851:26: note: candidate: ‘uint32_t lgfx::v1::color_conv_t::convert(const lgfx::v1::rgb565_t&)’
  851 |     LGFX_INLINE uint32_t convert(const rgb565_t&   c) { return convert_rgb565(  c.get()); }
      |                          ^~~~~~~
/home/rgault/Documents/PersonnalProjects/PaxOS-8/src/lua/../widgets/gui/../../interface/LovyanGFX/lgfx/v1/misc/colortype.hpp:852:26: note: candidate: ‘uint32_t lgfx::v1::color_conv_t::convert(const lgfx::v1::rgb888_t&)’
  852 |     LGFX_INLINE uint32_t convert(const rgb888_t&   c) { return convert_rgb888(  c.get()); }
      |                          ^~~~~~~
/home/rgault/Documents/PersonnalProjects/PaxOS-8/src/lua/../widgets/gui/../../interface/LovyanGFX/lgfx/v1/misc/colortype.hpp:853:26: note: candidate: ‘uint32_t lgfx::v1::color_conv_t::convert(const lgfx::v1::argb8888_t&)’
  853 |     LGFX_INLINE uint32_t convert(const argb8888_t& c) { return convert_argb8888(c.get()); }
      |                          ^~~~~~~
/home/rgault/Documents/PersonnalProjects/PaxOS-8/src/lua/../widgets/gui/../../interface/LovyanGFX/lgfx/v1/misc/colortype.hpp:854:26: note: candidate: ‘uint32_t lgfx::v1::color_conv_t::convert(const lgfx::v1::bgr888_t&)’
  854 |     LGFX_INLINE uint32_t convert(const bgr888_t&   c) { return convert_bgr888(  c.get()); }
      |                          ^~~~~~~
make[2]: *** [CMakeFiles/PaxOS.dir/build.make:902: CMakeFiles/PaxOS.dir/src/lua/lua.cpp.o] Error 1
make[1]: *** [CMakeFiles/Makefile2:83: CMakeFiles/PaxOS.dir/all] Error 2
make: *** [Makefile:91: all] Error 2

Here is the output of cmake .:

-- The C compiler identification is GNU 13.2.1
-- The CXX compiler identification is GNU 13.2.1
-- 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
> Building for Linux
-- Found CURL: /usr/lib/libcurl.so (found version "8.5.0")  
-- Configuring done (0.7s)
-- Generating done (0.0s)

I am on 058d654 on a clean clone.

Am I the only one getting this error ? Or is there a way to work around it ?

Thanks

@MopigamesYT
Copy link

I get the exact same thing.

@raphaelgault
Copy link
Author

Did someone have the chance to look at this ?
Thanks

ArthurPV added a commit to ArthurPV/PaxOS-8 that referenced this issue Feb 24, 2024
@ArthurPV ArthurPV linked a pull request Feb 24, 2024 that will close this issue
ArthurPV added a commit to ArthurPV/PaxOS-8 that referenced this issue Feb 24, 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

Successfully merging a pull request may close this issue.

2 participants