Warning: Arguments to path.join must be strings

[cedaro]

Warning: Arguments to path.join must be strings

[bradyvercher]

Interesting. I've run into that issue on other projects, but haven't experienced that while working on this one. Your config looks good.

Do you know what version of Grunt you have installed? Also what version of this package are you running? Oh, one more version, too: Which version of Node are you running?

[grappler]

"grunt": "~0.4.1" and "grunt-wp-i18n": "~0.2.0"

[bradyvercher]

Sorry, I made a quick edit to see what version of node you're running: node -v should do it.

[grappler]

v0.10.18

[bradyvercher]

Can you upgrade to the latest versions of node and Grunt and give it a shot? I'm running the latest on Windows, but will give it a go on OSX as well to see if I can replicate this.

[grappler]

I am on Windows 8.

[bradyvercher]

Actually, that's my fault. The readme has the wrong instructions. Try this config instead (notice the options object within the target):

makepot: {
    target: {
        options: {
            domainPath: '/languages',    // Where to save the POT file.
            mainFile: 'flowplayer.php',      // Main project file.
            potFilename: 'flowplayer5.pot',   // Name of the POT file.
            type: 'wp-plugin'  // Type of project (wp-plugin or wp-theme).
        }
    }
}

[grappler]

Great it works 😄 Do you also get PHP strict errors?

[bradyvercher]

Great! Sorry about that and thanks for bringing it to my attention.

The PHP strict errors are from the WordPress i18n tools. I could probably clean them up, but it would make maintaining the fork a little more difficult whenever changes are made upstream. In the meantime, I just disabled strict errors in the php.ini I use for the CLI.

[grappler]

Ok, I created a version that fixes most of errors. https://github.com/grappler/i18n

I tested it and it works