NOTICE: [boot] micha opened issue #150: The -C/--no-color boot option is ignored http://git.io/x2Ne
tbl has left the channel
NOTICE: [boot] micha pushed 1 new commit to master: http://git.io/x2Nz
NOTICE: boot/master 545a292 Micha Niskin: Fix issue where -C/--no-colors boot option was being ignored (fixes #150)
NOTICE: [boot] micha closed issue #150: The -C/--no-colors boot option is ignored http://git.io/x2Ne
tbl joined the channel
CookedGryphon has quit
CookedGryphon joined the channel
jk has quit
jk joined the channel
Viesti has quit
Viesti joined the channel
lance|afk is now known as lanceball
johann has quit
hayohayo joined the channel
johann joined the channel
AnxiousGarlic joined the channel
AnxiousGarlic has left the channel
piranha joined the channel
piranha has quit
dwn_ joined the channel
dwn_ has quit
jjttjj joined the channel
noprompt_ joined the channel
noprompt has quit
johann joined the channel
johann has quit
ul joined the channel
dm3 joined the channel
jjttjj has quit
johann joined the channel
johann has quit
piranha joined the channel
hayohayo has quit
noprompt_ has quit
noprompt joined the channel
noprompt
pandeiro: yes, i still maintain secretary. there's pr for the next major version but it hasn't been merged yet because i haven't had the chance to update the README for it.
piranha has quit
TheAncientGoat joined the channel
candiru has quit
johann joined the channel
piranha joined the channel
voytech joined the channel
schmir joined the channel
statonjr joined the channel
tbl has quit
johann joined the channel
johann has quit
iany has quit
tcrawley-away is now known as tcrawley
tbl joined the channel
pandeiro
martinklepsch: ping
martinklepsch
pong
pandeiro
for cljsjs libs that depend on other cljsjs libs, we need to both include the dependency and :requires, is that it?
eg react-router should have cljsjs.react 0.12.x in deps, and :require cljsjs.react as well?
(i just realized i haven't been including the dependency part, just the :requires)
martinklepsch
yup that sounds right
pandeiro
ok i gotta fix some packages :)
so this means that package A is pegged to a version of dep package B
(unless it's overriden in the application's top-level dependencies)
tcrawley is now known as tcrawley-away
martinklepsch
yes I think so
tbl has quit
pandeiro
yep ok makes sense
martinklepsch: thanks, that is all :)
martinklepsch
pandeiro: cool :)
pandeiro
Does Boot create "target" if it doesn't exist?
martinklepsch
pandeiro: yes
pandeiro: but only at the end of the cycle
pandeiro: (I assume you're asking because of these boot-http issues)
pandeiro
martinklepsch: yes exactly
martinklepsch: so is there any way boot-http could tap into that beforehand?
i guess not?
martinklepsch
pandeiro: don't think so
pandeiro
yeah, hmm
martinklepsch
alandipert: suggested just creating the directory that's passed to :dir — seems like the best thing to do to me
pandeiro
could boot not do that at the beginning instead of the end?
martinklepsch: `boot serve -d tagret`
tagret is created silently and served empty
nobody notices until you're demoing
:)
that is my concern with that
martinklepsch
well.. you can't account for typos really
pandeiro
you could say 'tagret doesn't exist, quitting'
or maybe special-case target
martinklepsch
pandeiro: maybe introduce an option to create the dir if it does not exist
pandeiro
is target the official target btw?
martinklepsch
pandeiro: it's the default
pandeiro
being the default maybe we can special case that one, otherwise exit 1 w/ an error
alandipert: ^-- sound sane?
johann joined the channel
or maybe stay away from special cases and make the directory but print a warning
michaniskin
yhe target can't be created eagerly though
it needs to be created lazily at the end
and only if there are files to put in it
pandeiro
michaniskin: ah ok, i'm sure you've thought about it
michaniskin: any take on the options above?
michaniskin
like should `boot -h` create a target dir?
pandeiro
michaniskin: ha
why not right?
michaniskin
tasks shouldn't be looking in there
the answer to the boot-http problem is to use temp dir instead of target
pandeiro
michaniskin: so don't actually serve the directory the user specifies, copy all of its contents to a tmpdir and serve that?
michaniskin
make a temp dir and in your task copy all output files to it
johann has quit
pandeiro
michaniskin: and for adhoc `boot -d pandeiro/boot-http serve -d . wait` too?
(there is no concept of an output file there, right?)
no build.boot, etc
michaniskin
right
but even then
maybe better would be
boot -r target -t /dev/null serve
boot needs a -T / --no-target option to set --target /dev/null
pandeiro
michaniskin: but 'better' in what sense?
for the end user, i think `boot serve -d target` is better, no?
michaniskin
but its error prone
pandeiro
yeah?
michaniskin
because you're defeating the pipeline model
pandeiro
yeah i see that
target is in flux
when using boot in a pipeline
michaniskin
so when you start 2boot processes like for a repl client
they fight over the target
things like that
pandeiro
yeah that makes sense
granted i haven't heard people mention that wrt http/serve 'in the wild'
have you hit that?
michaniskin
there are some issues in the discourse site about it
pandeiro
k i will have a look and study this further
michaniskin
maybe we should use file locking on the target dir