https://pantsbuild.org/ logo
Powered by
#general
  • e

    enough-analyst-54434

    08/31/2015, 8:47 PM
    @rough-minister-58256: is working on fs watching
  • e

    enough-analyst-54434

    08/31/2015, 8:48 PM
    we almost do - yes, not quite
  • w

    witty-crayon-22786

    08/31/2015, 8:48 PM
    hm, it's in master at least, heh. not hooked up to BUILD/sources, but
  • e

    enough-analyst-54434

    08/31/2015, 8:48 PM
    correct
  • e

    enough-analyst-54434

    08/31/2015, 8:48 PM
    I refer to the lack of hookup
  • w

    witty-crayon-22786

    08/31/2015, 8:48 PM
    k
  • e

    enough-analyst-54434

    08/31/2015, 8:48 PM
    thats not small!
  • e

    enough-analyst-54434

    08/31/2015, 8:49 PM
    by my guessing
  • e

    enough-analyst-54434

    08/31/2015, 8:49 PM
    nor is it huge
  • w

    witty-crayon-22786

    08/31/2015, 8:49 PM
    yea.
  • e

    enough-analyst-54434

    08/31/2015, 8:50 PM
    But - agreed, we should not need to erect a polling monstrosity, Kris's work will probably mesh timeline wise
  • w

    witty-crayon-22786

    08/31/2015, 8:50 PM
    Copy code
    1) fork `./pants changed-loop bundle.npm` (runs forever)
2) fork `./pants run.npm -- webpack-server` (runs forever)
3) iterate on changing sources
  • w

    witty-crayon-22786

    08/31/2015, 8:51 PM
    any alternative seems like it would involve support for killing the server on change and restarting it... probably cleaner, but more difficult
  • e

    enough-analyst-54434

    08/31/2015, 8:51 PM
    Well - if this is done right, there should only be 1
  • e

    enough-analyst-54434

    08/31/2015, 8:52 PM
    command, 
    ./pants bundle.npm --webpack-server
  • e

    enough-analyst-54434

    08/31/2015, 8:52 PM
    um -- with the changed.loop in there
  • e

    enough-analyst-54434

    08/31/2015, 8:52 PM
    since - IIUC webpack itself implements bundle
  • b

    bright-dinner-81701

    08/31/2015, 8:52 PM
    webpack-server?
  • e

    enough-analyst-54434

    08/31/2015, 8:52 PM
    I'm rolling with Stu's guess
  • b

    bright-dinner-81701

    08/31/2015, 8:53 PM
    i think we just want a directory that updates with changes to sources and targets
  • w

    witty-crayon-22786

    08/31/2015, 8:53 PM
    maybe the 
    changed-loop
    goal would work best (at least initially) as a process that just blocks until something has changed
  • w

    witty-crayon-22786

    08/31/2015, 8:53 PM
    (apparently that's how foursquare's impl works)
  • b

    bright-dinner-81701

    08/31/2015, 8:54 PM
    i dont think we want a —webpack-server, etc… js is too fragmented and changes too much to build in webpack specifically
  • b

    bright-dinner-81701

    08/31/2015, 8:54 PM
    we just want a rebuilding directory
  • e

    enough-analyst-54434

    08/31/2015, 8:54 PM
    I think thats the simplest thing to do with out re-arch pants - yes
  • w

    witty-crayon-22786

    08/31/2015, 8:54 PM
    @bright-dinner-81701: webpack already has a server.
  • e

    enough-analyst-54434

    08/31/2015, 8:54 PM
    sense any change to files owned by targets in context, re-run pants
  • w

    witty-crayon-22786

    08/31/2015, 8:54 PM
    i was referring to the existing webpack server (not sure how you launch it.)
  • b

    bright-dinner-81701

    08/31/2015, 8:54 PM
    ah ok
  • w

    witty-crayon-22786

    08/31/2015, 8:55 PM
    ok, so ... refactoring of the steps inbound
1...253254255...2420Latest