Laravel-mix: v0.8.3 mix.js outputs file relative to public folder

[adifaidz]

• Laravel Mix Version: 0.8.3
• Node Version : v6.9.1
• NPM Version : 3.10.8
• OS: Windows 10

Description:

The new v0.8.3 outputs the compiled javascript relative to public folder. So if now I were to compile this code,

mix.js('resources/assets/js/app.js', 'public/js')
   .sass('resources/assets/sass/app.scss', 'public/css');

It will output the files to:

public/public/js/app.js
public/css/app.css

Steps To Reproduce:

• Install v0.8.3 and run npm run dev
  

[michaellaw1701]

As a quick fix, I believe the error lies in the webpack path for public/js which Jeffery is probably fixing as I type, just removing the public part seems to work:

mix.js('resources/assets/js/app.js', '/js')
.sass('resources/assets/sass/styles.scss', 'public/css');

[Hendriksie]

I think this works as it should, probably because it now uses the publicPath variable.

If you look at the initialize function in the Mix.js:

if (this.isUsingLaravel()) this.publicPath = 'public';

I'm not sure if this is relevant but I thought it could be useful to tell you.

[adifaidz]

@weldricks yeah, forgot to point that out. Thanks 👍

[JeffreyWay]

I can't reproduce this. Can any of you do npm update laravel-mix to bump to 0.8.4, and then confirm that this is still an issue?

[d3radicated]

Can confirm in 0.8.4.

[JeffreyWay]

So if you do:

laravel new test && cd test
npm install
npm run dev

...you get public/public/js/app.js??

[d3radicated]

public/js/app.js   1.16 MB       0  [emitted]  [big]  public/js/app
    /css/app.css    684 kB       0  [emitted]  [big]  public/js/app

Yep, the built js goes to public/public/js/app.js.

[JeffreyWay]

Windows or Mac? You're positive that you're on 0.84? Fresh install gives me:

https://www.dropbox.com/s/szrtzb71n112bxx/Screenshot%202017-02-23%2011.25.28.png?dl=0

[d3radicated]

Windows 10. It might be an OS-specific issue.

Imgur

[ghost]

I can confirm this issue on a fresh laravel installation on Windows 10.

laravel/installer - 1.3.5
laravel-mix - 0.8.4
node - 6.10.0
npm - 4.2.0

[QWp6t]

I can confirm as well.

[JeffreyWay]

Okay, well it would be great if some of you Windows users could help out, and dig in to figure out what's happening here.

Maybe open this file and do a few console.logs() to figure out what the deal is. This is the file that builds up the Webpack entry points.

https://github.com/JeffreyWay/laravel-mix/blob/master/src/EntryBuilder.js#L40

The highlighted line in that file should remove the /public portion.

[d3radicated]

I added the following lines before this.entry.add().

console.log(paths);
console.log(this.entryName(paths.output));

[JeffreyWay]

@d3radicated Perfect. Can you go to that file, and change the entryName method to this?

    entryName(output) {
        return output.pathWithoutExt
            .replace(/\\/g, '/');
            .replace(this.mix.publicPath + '/', '/')
    }

And then run it again. Does that fix it? I bet it does.

[d3radicated]

That solves it.

[JeffreyWay]

Wahoo! I've fixed and tagged it. npm update laravel-mix. Thanks for the help. :)

[gags88]

Hi @JeffreyWay ...

Finally a long Day and i came across this post.
Degrading from v1.0 to v 0.8 worked for me.

But can you let me know that what is the alternative for
Mix.output().path in laravel-mix v1.0 that comes with laravel v5.4.22

as I am facing issues in below code with v1.0 with Mix.Paths.root and Mix.output().path

let HtmlWebpackPlugin = require('html-webpack-plugin');

module.exports.plugins.push(
  new HtmlWebpackPlugin({
    template: Mix.Paths.root('resources/views/index.html'),
    inject: false
  })
)

let PrerenderSpaPlugin = require('prerender-spa-plugin');
console.log(Mix.output().path);
module.exports.plugins.push(
  new PrerenderSpaPlugin(
    Mix.output().path,
    [ '/' ]
  )
)