The Wayback Machine - https://web.archive.org/web/20200915040313/https://github.com/google/shaka-player/issues/1929
Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

build.py default output path has changed #1929

Closed
asmirsabanovic opened this issue May 9, 2019 · 3 comments
Closed

build.py default output path has changed #1929

asmirsabanovic opened this issue May 9, 2019 · 3 comments
Assignees
Labels
Milestone

Comments

@asmirsabanovic
Copy link

@asmirsabanovic asmirsabanovic commented May 9, 2019

Have you read the FAQ and checked for duplicate open issues?
Yes

What version of Shaka Player are you using?
2.5.0

Can you reproduce the issue with our latest release version?
Yes

Can you reproduce the issue with the latest code from master?
Yes

Are you using the demo app or your own custom app?
custom app

If custom app, can you reproduce the issue using our demo app?
Yes

What browser and OS are you using?
Chrome (74.0.3729.131), OSX 10.14.2 (18C54)

For embedded devices (smart TVs, etc.), what model and firmware version are you using?

What are the manifest and license server URIs?

https://storage.googleapis.com/shaka-demo-assets/angel-one/dash.mpd

What did you do?

Run python build/build.py +@complete -@dash --force

What did you expect to happen?
dist/shaka-player.compiled.js to be built without dash support

What actually happened?

It was built with dash support and I could play a dash manifest. This used to work when I was using v2.4.7

@asmirsabanovic asmirsabanovic changed the title build.py exclude plugins not working [BUG]build.py exclude plugins not working May 9, 2019
@joeyparrish
Copy link
Member

@joeyparrish joeyparrish commented May 9, 2019

Because the demo app is built on top of the new Shaka Player UI, the default build has changed to the UI build in shaka-player.ui.js. To get the old behavior, you need to specify the name as "compiled" (default is "ui") and exclude the UI components (@ui). Try this:

python build/build.py +@complete -@dash -@ui --name compiled --force
@joeyparrish joeyparrish changed the title [BUG]build.py exclude plugins not working build.py default output path has changed May 9, 2019
@joeyparrish joeyparrish self-assigned this May 9, 2019
@asmirsabanovic
Copy link
Author

@asmirsabanovic asmirsabanovic commented May 9, 2019

It works now, thanks for the help and quick feedback! It might be a good idea to update the docs for v2.5.0: https://github.com/google/shaka-player/blob/master/build/README.md

@joeyparrish
Copy link
Member

@joeyparrish joeyparrish commented May 9, 2019

Yes! Thanks for pointing that out. I'll reopen this issue until we've corrected the docs.

@joeyparrish joeyparrish added docs and removed working as intended labels May 9, 2019
@joeyparrish joeyparrish reopened this May 9, 2019
@joeyparrish joeyparrish added this to the v2.6 milestone May 9, 2019
@shaka-bot shaka-bot closed this in c7ed9d7 Jun 11, 2019
TheModMaker added a commit that referenced this issue Jul 3, 2019
Closes #1929

Change-Id: Ie3a15ba3928ca21f9f4eb5c8ff423e2b5875782c
AnteWall added a commit to AnteWall/shaka-player that referenced this issue Jul 17, 2019
Closes google#1929

Change-Id: Ie3a15ba3928ca21f9f4eb5c8ff423e2b5875782c
@shaka-bot shaka-bot added the archived label Aug 10, 2019
@google google locked and limited conversation to collaborators Aug 10, 2019
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
3 participants
You can’t perform that action at this time.