search

hanatos / vkdt

raw photography workflow that sucks less
https://jo.dreggn.org/vkdt
BSD 2-Clause "Simplified" License
389 stars 36 forks source link

vkdt does not compile on Manjaro #127

Closed Heddesdorfer closed 5 months ago

Heddesdorfer commented 5 months ago

This is the log from "make -j20 debug > debug.txt" debug.txt vkdt not built, only vkdt-cli, vkdt-fit and vkdt-eval-profile

hanatos commented 5 months ago

interesting. obviously works here. your log doesn't even indicate that it tried to build the gui. do you have a custom config.mk? maybe something broke with the newer updates? is it a clean directory/fresh checkout? maybe bin/vkdt already exists and for odd reason has newer timestamp?

Heddesdorfer commented 5 months ago

I always follow these steps (I did it twice to sure): renamed directory vkdt to vkdtold $ git clone https://github.com/hanatos/vkdt.git --recursive $ cd vkdt $ git submodule init $ git submodule update copied config.mk from vkdtold/bin to vkdt/bin $ make -j20 debug > debug.txt all other known dependencies are installed.

config.mk.txt (added ".txt" to config, otherwise it couldn't be uploaded)

hanatos commented 5 months ago

hm okay sounds sane. could you try without your old config.mk? things moved a bit lately. you don't need a recursive checkout any more, there are no submodules (so these commands can go, too). also if you're configured with rawler you don't need to set the pugixml things in the config.

On Sun, Jun 16, 2024 at 8:35 PM Heddesdorfer @.***> wrote:

I always follow these steps (I did it twice to sure): renamed directory vkdt to vkdtold $ git clone https://github.com/hanatos/vkdt.git --recursive $ cd vkdt $ git submodule init $ git submodule update copied config.mk from vkdtold/bin to vkdt/bin $ make -j20 debug > debug.txt all other known dependencies are installed.

config.mk.txt https://github.com/user-attachments/files/15859446/config.mk.txt (added ".txt" to config, otherwise it couldn't be uploaded)

— Reply to this email directly, view it on GitHub https://github.com/hanatos/vkdt/issues/127#issuecomment-2171802586, or unsubscribe https://github.com/notifications/unsubscribe-auth/AAMAKKLR4RKILYCBX542WQTZHXLF3AVCNFSM6AAAAABJI3C5I2VHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDCNZRHAYDENJYGY . You are receiving this because you commented.Message ID: @.***>

Heddesdorfer commented 5 months ago

One thing I could do myself: I had to update rust from 1.78 to 1.79. But still no GUI was built. I appended compiler logs to file and console. vkdt_compilation_console_log.txt debug.txt

hanatos commented 5 months ago

.. do you have the xxd utility? it is needed to create the gui shaders.

On Mon, Jun 17, 2024 at 4:37 PM Heddesdorfer @.***> wrote:

One thing I could do myself: I had to update rust from 1.78 to 1.79. But still no GUI was built. I appended compiler logs to file and console. vkdt_compilation_console_log.txt https://github.com/user-attachments/files/15872744/vkdt_compilation_console_log.txt debug.txt https://github.com/user-attachments/files/15872746/debug.txt

— Reply to this email directly, view it on GitHub https://github.com/hanatos/vkdt/issues/127#issuecomment-2173594839, or unsubscribe https://github.com/notifications/unsubscribe-auth/AAMAKKMLCMG4TNLTUW2YRHLZH3YBNAVCNFSM6AAAAABJI3C5I2VHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDCNZTGU4TIOBTHE . You are receiving this because you commented.Message ID: @.***>

Heddesdorfer commented 5 months ago

This is a new dependency for me... Now vkdt compiles, but crashes. vkdt-bt-175265.txt

hanatos commented 5 months ago

okay you could try again with current master. in general these assertions are mostly validation layer complaints that are way more useful to debug with vkdt -d qvk because then the error message will be printed to the console. also these are very useful to debug/harden the program, and not very useful just to run it for productive use. in fact the debug build target introduces some extra slow checks and memory overhead..

Heddesdorfer commented 5 months ago

Now it works. Thank you!