Thanks, that helped On Mon, Jan 11, 2016 at 6:14 PM, Dave Brondsema <d...@brondsema.net> wrote:
> Works ok for me. I do remember seeing errors like that a while ago when > working > on the ticket though. There is a newer version of "babel" (which > translates the > ES6 to regular JS) released but Broccoli (which invokes babel and other > tasks) > doesn't support that new version yet. Perhaps npm got the wrong versions > somehow. Try deleting node_modules and tmp dirs and reinstall. See if > that helps? > > On 1/11/16 9:54 AM, Igor Bondarenko wrote: > > I'm getting an error running `npm run build` inside docker on a fresh > > master (3424bee). > > > > $ dc run web npm run build > > > >> allura@0.0.0 build /allura > >> rm -rf Allura/allura/public/nf/js/build/; broccoli build > > Allura/allura/public/nf/js/build/ > > > > File: context-menu.es6.js > > SyntaxError: context-menu.es6.js: Unexpected token (27:21) > > 25 | } > > 26 | > >> 27 | static propTypes = { > > | ^ > > > > > > Anyone knows why is that? > > > > Versions of nodejs and npm I have installed: > > > > $ dc run web bash -c 'node -v && npm -v' > > v4.2.2 > > 2.14.7 > > > > > > On Sat, Jan 9, 2016 at 1:00 AM, Dave Brondsema <d...@brondsema.net> > wrote: > > > >> For folks doing current development, the master branch now requires some > >> `npm` > >> commands to be run. The admin navbar code merged from > >> https://forge-allura.apache.org/p/allura/tickets/7919/ starts using JS > >> files > >> that are written in ES6 and JSX syntax, so we have commands to transpile > >> them > >> into regular JS. You'll need to run these for the admin nav bar to > work, > >> which > >> is now the way to add & edit tools. > >> > >> To install nodejs & npm, and run the commands, see the Installation > >> instructions: > >> > >> > https://forge-allura.apache.org/docs/getting_started/installation.html#a-few-more-steps-if-using-git > >> > >> You'll probably need to run that just once (unless you delete the > >> transpiled.js > >> file it creates). If you are editing the .es6.js files and want to > >> transpile > >> them frequently during development, see the npm commands in this > section: > >> > >> > https://forge-allura.apache.org/docs/development/contributing.html?highlight=broccoli#technology-stack > >> > >> -- > >> Dave Brondsema : d...@brondsema.net > >> http://www.brondsema.net : personal > >> http://www.splike.com : programming > >> <>< > >> > > > > > > -- > Dave Brondsema : d...@brondsema.net > http://www.brondsema.net : personal > http://www.splike.com : programming > <>< >