Default phases

[Hugo Duncan]

Many of the new 0.8 crates are using an :install phase to actually
install the relevant phase, and using the :configure phase to just
create configuration files, etc.

The :install is however not run by default, which can break the
principle of least surprise.

I think there are several possibilities for solving this.

i) we could add :install as a default phase
ii) we could allow server-spec's to declare a default phase list
iii) we could run all phases by default

I tend towards ii), as it would be the most flexible. We could allow
:extends to combine default phase lists by imposing some sort of total
ordering that does not contradict any of the extended server-spec's
default phase lists (or error if a total ordering can not be found).

The problem I see with i), is that it adds more special case knowledge
that you need to know to use pallet, and imposes the standard phases on
all pallet users.

The problem I see with iii) is that the phases are defined in a map and
have no intrinsic ordering (yes, I know ordered maps exist).

Another possibility would be to combine iii) with ii).

Thoughts? Any other suggestions?

Hugo

[Bruce Durling]

Hugo,

I think ii (define a list for each server) would work best for me
though I'd want to be able to define a new list at the top level
server-spec if I have a spec that extends multiple servers. If
specified at the top level it should override the extended servers
specs as I might need to change the order of the phases depending on
how things interact

This is less of a problem with bootstrap/settings/configure/install as
they shouldn't really interact yet (except perhaps install) and then
they just need to run left to right, which is what they already do
according to the order of server-specs in the :extends section

cheers,
Bruce

> --
> You received this message because you are subscribed to the Google Groups "pallet" group.
> To unsubscribe from this group and stop receiving emails from it, send an email to pallet-clj+...@googlegroups.com.
> For more options, visit https://groups.google.com/groups/opt_out.
>
>



--
@otfrom | CTO & co-founder @MastodonC | mastodonc.com
See recent coverage of us in the Economist http://econ.st/WeTd2i and
the Financial Times http://on.ft.com/T154BA