search

OpenXT / openxt

Base OpenXT build scripts
48 stars 39 forks source link

do_build.sh improvements #342

Closed jandryuk closed 5 years ago

jandryuk commented 5 years ago

Drop sysroot from the script. It's unneeded and unused as far as I can tell. I usually drop syncui,source,sdk,license,sourceinfo as well, but I've left those.

Re-order sourcing the .config file before parsing the command line options. This way you can set your common variables in .config and override on the command line. The current way does not allow command line overriding. For instance, you can set your default STEPS in .config, but then run just ./do_build.sh -s ndvm,ship to do a quicker partial re-build.

Third, print the date when finished. I find this useful when I have multiple windows open to know when a build last finished.

eric-ch commented 5 years ago

Will merge soon.

OXT-1548: Should that image be deprecated? Is anyone still using/planning to maintain it?

jandryuk commented 5 years ago

@dpsmith and I talked about removing the sysroot image. We're not aware of any use for it. It may also be broken anyway:

It adds watch_surfman to the inittab: xs:5:respawn:/usr/bin/watch_surfman

In a quick search, I can't find watch_surfman.