In install_hue.sh there are 2 variables related to logs: LOG_DIR and DESKTOP_LOG_DIR. While they may be required by the environment, it may be cleaner to keep it to one.
That's just a convention that we've got going in our _install.sh scripts. I agree that it is a bit confusing since Hue has the same var, but it is probably best to stay consistent within the _install.sh scripts.
In hue-app-postinstall.tpl, hue-app.prerm.tpl and other app related files kept some variables that are removed in other files, such as ROOT=/usr/lib/hue
That's on purpose. We need this functionality to remain for things that are plugins into Hue. The model of Hue integration in Bigtop is that we prepackage all the core bits together (and thus do not need to register them dynamically) but things like beeswax and future plugins still need to run registration code at package install time.
In a few places user name "hue" is used without defining USER=hue
Could you, please, be a bit more specific? I could only see that in chown lines which could arguably be changed.
Tried a few commands. Met this problem: /usr/lib/hue/build/env/bin]$ sudo ./hue test
In general Bigtop packaging tries to remove test code from packages. Hence any residual code is not expected to work. An argument could be had that we should do a better job of completely removing hue test code, but it looks pretty tightly integrated. Do you want to take a look at it?
Off-topic question: why the build system does not detect change in the src files, only `rm -rf <component>/build`?
Which build system are you referring to? The Bigtop itself or Hue? Can you, please, be more specific?