Add psvDebug, Ctrl, Touch samples #5

Merged
merged 11 commits into from Aug 18, 2016

Projects

None yet

2 participants

@yne
Contributor
yne commented Aug 17, 2016

This PR add the following:

  • Codebase cleanup
  • Add basic ANSI escape support to the psvDebug library (no bold, underline, blink support)
  • Add psvDebug sample (with colorization, and placement)
  • Add sceCtrl sample
  • Add sceTouch sample
@yne yne Fix invalid build.sh directory
The net/http sample were not built since the net/ directory were mistaken for a sample directory
3a4b716
@xyzz xyzz and 1 other commented on an outdated diff Aug 18, 2016
@@ -1,13 +1,20 @@
# vitasdk code samples
-Every directory contains a single sample. To build all samples, add the toolchain `bin/` directory to your `$PATH`, then run `./build.sh`. To clean all samples, run `./clean.sh`. To build a single sample, `cd` to its directory and type `make`.
+## Prerequists
@xyzz
xyzz Aug 18, 2016 Member

Prerequisites

@yne
yne Aug 18, 2016 Contributor

Yup, sorry

@xyzz xyzz commented on the diff Aug 18, 2016
README.md
-* `hello_world`: A minimal Makefile-based C project
-* `hello_cpp_world`: A minimal Makefile-based C++ project
+Every directory contains a single sample. `cd` to its directory and use the `make` command to build it.
+
+## Building everything
+
+Use the following command to build every samples
+
+```
+for f in */Makefile; do make -C ${f%/*} all; done
@xyzz
xyzz Aug 18, 2016 Member

this is outdated now, just run build.sh?

@yne
yne Aug 18, 2016 Contributor

As said on #4

Basically, when this repository will have dozens of samples (I hope it will), I seriously doubt that people will build all of them. They will simply go the the sample they are interested in an build-install-run it.

@xyzz xyzz commented on the diff Aug 18, 2016
README.md
-* `hello_world`: A minimal Makefile-based C project
-* `hello_cpp_world`: A minimal Makefile-based C++ project
@xyzz
xyzz Aug 18, 2016 Member

Why did you delete this? It would be nice to have short sample descriptions here, with optionally a longer README.md in the sample directory.

@yne
yne Aug 18, 2016 Contributor

Because nobody will remember to update the main README when they will add/change a sample (for example, the net_http sample were already missing ;-) ).

👍 for the per-sample README

We could just explain in this main README that every samples have they own README
So we don't have to update the main one each time.

@xyzz
Member
xyzz commented Aug 18, 2016

ok so just fix the typo and I'll merge it

@yne
Contributor
yne commented Aug 18, 2016

I've fixed the typo, and added a new RTC sample to match the vita-header addition.

@xyzz xyzz merged commit 19f7f0d into vitasdk:master Aug 18, 2016
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment