Patch to compile less when minifying files - updated logic

[tobes]

This is a sample patch to compile less css files when minifying css/js.

It uses node to compile but makes it part of the minification process.

In my mind the two should be done at the same time so one is not forgotten probably the pater command needs to be renamed. Also I'd be inclined to make it more specific so that we do not specify the directory for minification as we want public/base

I'm happy if people want this done differently.

It also creates the 'custom colour styles'

currently error checking is week

[johnmartin]

@tobes This is good. Just one thing:

Would it be possible to move the less methods into their own paster command? Then have another paster command that was something like front-end-build which builds the less then minifies?

[tobes]

yeah I can do that not sure on the name but that will do for now

[johnmartin]

@tobes What's the status on this patch? (I'd like to write the ckan 2.0 front end deploy process for @amercader)

[tobes]

yeah I forgot about this it's on my todo list now

[johnmartin]

@tobes Ping.

[tobes]

@johnmartin

we now have paster less and paster front-end-build which also does the js translation files

ready for re review

[johnmartin]

Should be something like:

'''Compile all root less documents into their CSS counterparts

[johnmartin]

OK, does this mean we can get rid of main.debug.css (I don't really understand why we have it)? and ditch ./bin/less?

[tobes]

thanks,

I'll fix the cut paste bugs :)

with main.debug.css the point of this is that it is a none git controlled file that is updated via bin/less (which should be renamed IMHO) . the idea is that you especially when playing around with the css via .less files will see the latest css whilst doing development without seeing the noise in you git diffs etc. It is a bit like the minification stuff where we do not want to do it all over the place because it just causes noise and merge conflicts that help no-one. Now there may be a more graceful way of doing this and if so changing would be ok by me. This is stuff that Aran created because it worked well for him. Before this we were using some js lib to compile the less files but this was too slow to be a usable option.

If you need more info we can discuss on irc or skype :)

[tobes]

@johnmartin fixed docstring

[johnmartin]

@tobes This is good. Except on thing:

I was getting an error with running paster front-end-build. Essentially polib wasn't in my environment (see https://github.com/okfn/ckan/blob/master/ckan/lib/cli.py#L1676)... looking at pip-requirements.txt it's not in there either.

Is this a bug in master? Or is it something odd with my environment?

[tobes]

this is a separate issue it is needed for development/deployment but not for using
we should maybe get a better fix like pip-dev.txt

[johnmartin]

@tobes Perfect. I'll merge this.