waywardgeek / infnoise

The world's easiest TRNG to get right
Creative Commons Zero v1.0 Universal
732 stars 100 forks source link

Update Makefile #56

Closed ghost closed 6 years ago

ghost commented 6 years ago

In Manjaro libftdi-config is not available, libftdi1-config is. After this 'make install' works, but 'systemctl start infnoise' waits and after some time results in: infnoise.service: Job infnoise.service/start failed with result 'dependency'. dev-infnoise.device: Job dev-infnoise.device/start failed with result 'timeout'.

I have no solution for this, except going to a previous build.

B.t.w. I closed this because I thought I was too fast, you were still working on it.

manuel-domke commented 6 years ago

Why did you close it?

Sorry for the broken build on Manjaro. I have automated tests on Arch, but seems like thats no enough ;-)

Unfortunately libftdi1-config is again not available on the other distros, so it may be better so stick to the previous approach - but using /sbin/ldconfig to make it work everywhere. Included in #55.

FOUND = $(shell /sbin/ldconfig -p | grep --silent libftdi.so && echo found)
ifeq ($(FOUND), found)
        FTDI=   -lftdi
else
        FTDI=   -lftdi1
endif

Whats been the solution to your 'systemctl start infnoise' problem? It should not do that at all - and libinfnoise.so is not a dependency (yet). This more likely seemed to be a problem with the udev rule?

manuel-domke commented 6 years ago

I've switched to hardcode /sbin/ldconfig to make it work anywhere I've tested so far. In #58

ghost commented 6 years ago

I tested it but 'systemctl start infnoise' still does not return to a prompt under Manjaro. However I discovered that when commenting out 'BindsTo=dev-infnoise.device' in infnoise.service, it works ok. Maybe hardcoding /sbin/ldconfig was not necessary. There is no 'dev-infnoise.device' present on my system. Could that be the cause of the problem?