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

Readme.md update #121

Closed
wants to merge 174 commits into from
Closed

Readme.md update #121

wants to merge 174 commits into from

Conversation

psiphi75
Copy link
Contributor

@psiphi75 psiphi75 commented Apr 3, 2016

... more to come.

adityapatadia and others added 23 commits June 22, 2015 23:16
…rror

fixed exception handler for dtc compile command
There are some potential areas that could cause errors.   These are:
  - Line 21: "err" used to be hoisted to global scope.
  - Lines 132/179/203: "files" is already in scope, it's defined in the parent function.

Some unused global variables are commented out.
Fix potential errors due to JavaScript scope
Now you can run octalbonescript like this

    AUTO_LOAD_CAPE=0 node myscript.js

This means that capes won't be automatically loaded.

Now you can run octalbonescript like this

AUTO_LOAD_CAPE=0 node myscript.js

This means that capes won't be automatically loaded.
Added option to disable autoloading of capes at startup
@adityapatadia
Copy link
Contributor

I tried this long ago. @jadonk is not willing to merge, that's why a fork is created and maintained by me.

@psiphi75
Copy link
Contributor Author

psiphi75 commented Apr 4, 2016

...doh, sent a pull request to the wrong branch.

@psiphi75 psiphi75 closed this Apr 4, 2016
@jadonk
Copy link
Owner

jadonk commented Apr 17, 2016

@adityapatadia I'm willing to merge changes, just not sweeping changes that break things I'm trying to do.

@psiphi75
Copy link
Contributor Author

@jadonk as an active user of octalbonescript I think it would be great to see octalbonescript (OBS) merge back into bonescript. Although I see that there would breaking changes if the merge were to happen as it is now. I believe that it would be possible to merge and reduce the number of breaking changes.

The reason I use OBS over bonescript is that it works with more recent kernels and is more intuitive when coming from a node.js background (asynchronicity and standard callbacks) and I have found it to be more stable.

I would be glad to discuss in more detail and help in the merge process (dev/testing).

@jadonk
Copy link
Owner

jadonk commented Apr 18, 2016

@psiphi75 that'd be great. I'm happy to comment against a pull request. With the 0.5.x branch of BoneScript, support is now there for the 4.1 kernels. We simplified it to use the pinmux helpers rather than load device tree fragments. Moving forward, I'd like to rely on pinmux helpers and provide ways to help users ensure a suitable pinmux helper is installed for the pins they want to use.

@psiphi75
Copy link
Contributor Author

@adityapatadia: I have added a couple of feature requests to @jadonk for bonescript. See:

Let me know if you have any comments.

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.

6 participants