petejkim / autoparts

package manager for nitrous.io
BSD 2-Clause "Simplified" License
81 stars 84 forks source link

Check for updates when starting a new shell. #153

Open laktek opened 10 years ago

laktek commented 10 years ago

This reinstates the original behaviour of checking for updates (which is included in parts init) when starting a new shell.

dqminh commented 10 years ago

We removed parts init for a reason https://github.com/nitrous-io/autoparts/commit/c27b21355fde53db4244ff4952de339f8648c45e

Can you make sure that we can auto-update with parts env instead of reusing parts init, maybe with parts env --update or something ?

laktek commented 10 years ago

But there's a following commit https://github.com/nitrous-io/autoparts/commit/ca7839a8e386dfac63517b3dfeb51a40d1aabea9, which prevents parts init - from auto-starting packages (reason stated in the commit that removed parts init -).

So I guess it's safe to use parts init - again.

chuyeow commented 10 years ago

I think this is fine except I'm not sure if we need to call parts init - inside script/setup_buildenv.

IMHO, the interface should be like this so that there are no surprises:

But we don't have to change this.

laktek commented 10 years ago

Before https://github.com/nitrous-io/autoparts/commit/c27b21355fde53db4244ff4952de339f8648c45e, script/setup_buildenv also called parts init -. So basically this reverts to the original behaviour (not really sure whether calling parts env there is fine).