Skip to content

Conversation

@r3pwn
Copy link

@r3pwn r3pwn commented Aug 10, 2020

This is a rebase of the "--serve commits" ( #1419 ) onto the latest master branch, with all merge issues resolved.

Still seems to be a bit buggy, but this at least puts it on the most recent master branch cleanly

@r3pwn r3pwn changed the title Rebase of --serve commits onto v1.16 [wip] Rebase of --serve commits onto v1.16 Aug 10, 2020
@r3pwn
Copy link
Author

r3pwn commented Aug 10, 2020

I'm using @rom1v's suggestion of testing using VLC and it does appear to work for a bit. I had it crash on me once, but haven't been able to reproduce. Additionally, the device window is also visible, which I would classify as a bug. Looking into it

@rom1v
Copy link
Collaborator

rom1v commented Aug 10, 2020

Rebase of …

Good idea 👍

Merge branch 'master' of github.com:Genymobile/scrcpy

You should never merge a branch in a PR (just rebase instead) 😉

Killian Richard added 4 commits August 10, 2020 19:59
This commit combines the following commits:

- Adding --serve option on README
- Adding --serve option : Add current working files
- Update --serve option with scrcpy v1.13 + Cleaning code.
- Fix missing char and error
- Allowing other ip address by converting it to hex

Rebased on top of the v1.16 source
@r3pwn
Copy link
Author

r3pwn commented Aug 11, 2020

@rom1v My bad, I have resolved that and properly rebased it

@r3pwn r3pwn changed the title [wip] Rebase of --serve commits onto v1.16 Rebase of --serve commits onto v1.16 Aug 11, 2020
@r3pwn
Copy link
Author

r3pwn commented Aug 11, 2020

After some more testing, I haven't been able to reproduce that crash I initially saw, and I realized if I want the display to not appear, it's possible to pass in --no-display

@r3pwn
Copy link
Author

r3pwn commented Sep 2, 2020

@rom1v Have you had the chance to take a look at this PR?

@rom1v
Copy link
Collaborator

rom1v commented Jan 16, 2022

Please read #1419 (comment)

@rom1v rom1v closed this Feb 22, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants