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

Port http://ph7.symisc.net/ at 29f589677ee88c8958237c4a02d212d59d8b6106 #81

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

JayFoxRox
Copy link
Owner

Taken from https://github.com/symisc/PH7/tree/29f589677ee88c8958237c4a02d212d59d8b6106

A rubbish port, done in a couple of minutes.

This should be moved into a separate repo that forks the original, for a cleaner diff and exposure.

@JayFoxRox
Copy link
Owner Author

  • The wide-string changes should probably be upstreamed.
  • Some features should be made optional for better portability.
  • The header in ph7.c contains a lot of unfinished winapi functions, those need fixing for many features to work properly.

"echo 'Welcome guest'.PHP_EOL;"\
"echo 'Current system time is: '.date('Y-m-d H:i:s').PHP_EOL;"\
"echo 'and you are running '.php_uname();"\
"?>"
Copy link
Owner Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This should add a final EOL and also sleep for a bit.
On desktop the output looks ugly otherwise. On Xbox it just keeps rebooting way too fast, so the text never really shows up.

Copy link

@GXTX GXTX Jul 11, 2020

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Things like fatal errors will also immediately reboot the Xbox as well. Maybe the sample should instead Sleep(x) before exiting.

Copy link
Owner Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I see this as a small demo, but ideally it would be a lib and people would handle it however they want in their app (either by adding an error handler to their VM, or by hardcoding "good" code like this sample does).

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