Add option to specify destination directory for taks

[skotchio]

It would be cool if you add option to specify destination directory for taks like the following:

jadeUsemin: {
    scripts: {
        options: {
            tasks: {
                js: ['concat', 'uglify'],
                css: ['concat', 'cssmin']
            },
            taskDestDir: './build'
        },
        files: [{
            dest: './src/partials/index.jade',
            src: './public/partials/index.jade'
        }]
    }
}

[pgilad]

Did you try using the options targetPrefix ?

What Dest directory are you referring to? the one that the compiled css/js gets compiled to or the ones for the destination jade file?

[skotchio]

@pgilad Thanks for the help, options targetPrefix is that I need 👍

[skotchio]

It's seem I have found a bug when using options targetPrefix provided. On the following config:

    jadeUsemin: {
        vendor: {
            options: {
                tasks: {
                    js: ['concat', 'filerev'],
                    css: ['concat', 'filerev']
                },
                targetPrefix: '.tmp',
                dirTasks: 'filerev'
            },
            files: [{
                    expand: true,
                    cwd: "src/to/views",
                    src: ['*.jade'],
                    dest: 'build/views'
                }
            ]
        }
    }

Paths In the compiled views added without filerev hashes.

[pgilad]

Can you try running grunt jadeUsemin:vendor --verbose?

[skotchio]

@pgilad unfortunately I can't show my technical detail for all. Please could you try my config yourself?

[pgilad]

I could but I don't have your dir file structure and/or your files...
Also you are getting targetPrefix options wrong. Why do you want to output compiled files there?

How do you run your task?

[skotchio]

Why I'm using targetPrefix option wrong? A need to reside files to .tmp directory first because I use google closure compiler for my js files but it does't support destination directory as source one.

I run my task via Gruntfile.

[pgilad]

I'm going to need either a verbose log (run with grunt jadeUsemin --verbose) or access to the files, since I can't use your config without the files.

Perhaps try to create a minimum test case where the task fails so that this can be debugged.

The targetPrefix + filerev have been successfully used by others & me, and also have passing tests for them, so either it's an edge case that doesn't get handled right or you might be using it wrong

[ppitonak]

Hi, I had the same error as @skotchio that paths in the compiled views had no filerev hashes.

The problem was caused by missing / at the end in prefix and targetPrefix. @pgilad you can try this by removing slash in Gruntfile.js: https://github.com/pgilad/grunt-jade-usemin/blob/master/Gruntfile.js#L69-L70 ... tests will fail.

[ppitonak]

It's a problem with targetPrefix, when only prefix doesn't contain slash, tests pass.

[pgilad]

Hi, thanks for the discovery, can you create a failing PR test case for this (and a possible fix)?
I think we can easily make sure prefix contains an ending slash...

[pgilad]

Fixed and thanks to @ppitonak for the help...