ng update @angular/cli with `allowOutsideOutDir ` not working - Path is Invalid

[johnpapa]

Versions

Angular CLI: 6.0.0
Node: 8.11.1
OS: darwin x64
Angular: 6.0.0
... animations, cli, common, compiler, compiler-cli, core, forms
... http, language-service, platform-browser
... platform-browser-dynamic, router

Package                           Version
-----------------------------------------------------------
@angular-devkit/architect         0.6.0
@angular-devkit/build-angular     0.6.0
@angular-devkit/build-optimizer   0.6.0
@angular-devkit/core              0.6.0
@angular-devkit/schematics        0.6.0
@angular/cdk                      5.2.4
@angular/material                 5.2.4
@ngtools/webpack                  6.0.0
@schematics/angular               0.6.0
@schematics/update                0.6.0
rxjs                              6.1.0
typescript                        2.7.2
webpack                           4.6.0

Repro steps

• create an angular app with a src of /src/client
• create a folder src/server (this is where i could put a node server)
• add this to your package.json inside of apps, assets.

        {
          "glob": "**/*.*",
          "input": "../server/",
          "output": "../",
          "allowOutsideOutDir": true
        },
        {
          "glob": "package.json",
          "input": "../../",
          "output": "../",
          "allowOutsideOutDir": true
        }

• run ng update @angular/cli

Observed behavior

▶ng update @angular/cli
            Updating karma configuration
            Updating configuration
Path "/../" is invalid.

Mention any other details that might be useful (optional)

I removed those allowOutsideOutDir entries and it seems to work. But I think it should work with them.

cc @filipesilva

[johnpapa]

it seems allowOutsideOutDir has been removed.

I recommend someting in the changelog, the upgrade guide, and a mlog message in the console when running the update command ... thise wil help folks know whats up

i do not see any reference to it in the changelog. but i see this issue #9363

Any guidance is appreciated.

[filipesilva]

I think we should both show a warning on the release notes, and also show a warning during ng update about those assets entries.

[filipesilva]

@devarg I don't think your comment is related to the issue shown here by @johnpapa. Can you instead open a new issue? Thanks!

[filipesilva]

Added a Breaking Changes section to https://github.com/angular/angular-cli/releases/tag/v6.0.0, with this change listed.

[devarg]

@filipesilva I'm sorry you are right, build the issue has been moved to to issue 10649

[elvisbegovic]

@johnpapa can you please search before open issue #10350

It is not first time you open this one #8122

While this already exist #7773

Thanks