OK, it would be cool to have a drush command that forces a (re)processing of a list of files.

It would form part of a chain which could come together to solve #1539278: Live compilation.

Patch to follow.

Comments

jamsilver’s picture

Assigned: jamsilver » Unassigned
Status: Needs work » Needs review
StatusFileSize
new3.47 KB

Note this patch depends on the patch in #1602174: Add support for JS preprocessing - well at least it depends on the existance and signature of a function called _prepro_process_assets().

richthegeek’s picture

Status: Needs review » Fixed

Added in commit #98a5378, will be in 0.5 when that gets released later today. thanks for the help :)

Status: Fixed » Closed (fixed)

Automatically closed -- issue fixed for 2 weeks with no activity.

enrique.delgado’s picture

Howdy,

I'm having a hard time figuring out how to run the drush command. I've tried passing in the full path to a .scss file, but I keep getting this drush warning:

No files were matched by current prepro configuration.

After looking at the prepro configuration, I don't see anything that seems like filepath in there. This is a screenshot of my config screen for prepro, nothing much to it: https://skitch.com/edelgado-digium/e1767/preprocessors

What am I missing? Where do I setup the filepaths that prepro is expecting? I'm using Sassy PHPSass as the pre-processor as you can see. There is nothing in that module that I can see where I can specify these file paths either.

Any help would be greatly appreciated.

enrique.delgado’s picture

Anyone?

joseph.olstad’s picture

Issue summary: View changes

looks like the drush functionality was removed from the later versions of prepro.

We'd have to re-apply it.

joseph.olstad’s picture

Version: 7.x-0.x-dev » 7.x-1.x-dev
Status: Closed (fixed) » Needs work
sense-design’s picture

Could you please provide a complete patch for 7.x-1.x branch? I will then add the patch to the branch for reviewing.

  • joseph.olstad authored d93ac45 on 7.x-1.x
    Issue #1614346 by jamsilver, joseph.olstad: Drush integration, add...
joseph.olstad’s picture

Status: Needs work » Fixed
StatusFileSize
new1.36 KB

The first patch was no longer working, and it seems just easier to clear the prepro caches.

see patch and commit.

joseph.olstad’s picture

joseph.olstad’s picture

If you're using sassy with prepro, the new command 'drush pcc' will clear the prepro caches and also clear the css caches, otherwise 'drush pcc' will clear just the prepro caches.

sense-design’s picture

Nice work

joseph.olstad’s picture

Status: Fixed » Closed (fixed)

Automatically closed - issue fixed for 2 weeks with no activity.

joseph.olstad’s picture

Status: Closed (fixed) » Needs review

Inadvertantly used hook_clear_cache instead of a custom function, now on any type of hook_clear_cache call using drush even a drush cc css-js the prepro cache gets cleared.

Had meant this to be a seperate command with as drush pcc (prepro cache clear) rather than any old cache-clear . So perhaps the functionality should be changed, say, cache clear only when drush cc all (all caches) are called, or add a seperate handler for prepro , plus leave the specific one drush pcc intact.

This change should be easy to do. any feedback?