Update CompileHowto.txt #706

Merged
merged 5 commits into from Jul 13, 2016

Projects

None yet

4 participants

@NicoHood
Contributor

I am not sure if a python dep is really required and for which purpose.

I think we should add the deb package information and how to manually enable the systemd service. I am going to look at the AUR packages soon.

@penfold42
Contributor

The effect engine uses Python - they're all Python scripts

@NicoHood
Contributor
NicoHood commented Jun 28, 2016 edited

Should I install python (aka 3) or python2 for this?

Also it seems icu is also required:

Unable to load library icui18n "Cannot load library icui18n: (icui18n: cannot open shared object file: No such file or directory)"
@penfold42
Contributor

Both work ok for nearly everyone.

My clock effect crashes with python 3 but that's the only issue so far

@NicoHood
Contributor

Thanks. So is the package python okay or should I also include some dev libraries for python? (did use python that much so far).

@redPanther
Contributor

instead of:
cmake -DENABLE_WS2812BPWM=ON -DENABLE_WS281XPWM=ON -DCMAKE_BUILD_TYPE=Release -DPLATFORM="rpi" -Wno-dev ..

use
cmake -DCMAKE_BUILD_TYPE=Release -DPLATFORM="rpi-pwm" -Wno-dev ..

read main cmakelists for details

@NicoHood
Contributor

Thanks. Arent the other lines also out of date? Like line 33 for orange pi. I guess spi is disabled by default then? Also a line for x64 linux would be nice to add.

@redPanther
Contributor

X86 32 and 64 bit are the same on that level of abstraction

@NicoHood
Contributor

I meant arm vs intel. So you know which options you have to use when compiling for a normal pc. That is unclear to me, as there is only an instruction for orange pi. Maybe its the same, maybe you do not need any of those options. Which cmake params would be best on a normal laptop (which uses an usb device as output)?

@NicoHood
Contributor
NicoHood commented Jul 2, 2016 edited

I just checked the AUR pkgbuild for hyperion on desktop. They do not use the protobuf submodule, but rather the one from the official repository (which is a good idea).
https://aur.archlinux.org/cgit/aur.git/tree/PKGBUILD?h=hyperion
@patlux

Can't we make an if/else for building on x64 and not use the submodule? Or why is the submodule even required? Can't we use the repos? Maybe this version is outdated now and no longer needed.

@redPanther
Contributor

I,ve read tge aur. This is absolutly.outdated. install merhod, git url, cmake flags ... needs an update. Especialy for next hyperion

@brindosch
Collaborator
brindosch commented Jul 2, 2016 edited

Now i know how we should fix issues!

sed -i 's/isnan/std::isnan/g' libsrc/leddevice/LedDevicePhilipsHue.cpp

;)

@NicoHood
Contributor
NicoHood commented Jul 2, 2016

@redPanther ah I see. Never heard of that before. Seems I have to check the other repository now and open PRs over there.

@brindosch what? This is totally unrelated?

@brindosch
Collaborator
brindosch commented Jul 2, 2016 edited

Please don't add a arch issue over there. There is enough major stuff open currently. We are far away from this step.

@brindosch
Collaborator
brindosch commented Jul 2, 2016 edited

And additional nothing is final, like dependencies list.

@redPanther
Contributor

Open an issue at aur. The creator of the aur script should fix this. He can make a new script (hyperion.ng-git) and make all changes. We can guide him. Atm we (hyperion team) had handsfull with lots of things. If you want you can establish the contact and convince him to make a new aur script - that would be helpfull in this case.
Best way for such discussion is in hyperion forum in the develpment section.

@NicoHood
Contributor
NicoHood commented Jul 3, 2016

I see. I am going to prepare an AUR package myself. When is the first release of NG expected (as a tag would be nice)?

@brindosch brindosch merged commit ec324f1 into hyperion-project:master Jul 13, 2016
@brindosch brindosch modified the milestone: V1.03.1 Jul 24, 2016
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment