Skip to content

Conversation

@lmoellendorf
Copy link
Contributor

This will add installation instructions for NQC on Linux.

From: bricxcc[at]comcast[dot]net
To: 'Lars Möllendorf'
Cc:
Bcc:
Subject: RE: Include Tutorials and Samples into maintained fork on Github
Reply-To:

All of the materials that are part of NQC and NXC and BricxCC are covered by the MPL. You are welcome to add them to the github repo.

https://bricxcc.sourceforge.net/nqc/doc/index.html
@mesheets
Copy link
Member

Appreciate the engagement and the pull request!

The first two commits with the Linux installation instruction updates look like they should be fine, but the third commit appears to duplicate what is available under the NQC-Libs project here (linked to in the alert "Tip" callout on the main NQC GitHub page). Not everyone seems to see that link, so I'm open to ways to better highlight the relationship between the two projects. :-)

The initial library content was originally separate, and part of the intent might have been to separate internal content needed to build NQC from user-facing content intended for those just wanting to program with NQC.

Thank you!

...as they are available under the
[NQC-Libs](https://github.com/BrickBot/nqc-libs) project
[here](https://github.com/BrickBot/nqc-libs/tree/master/tutorial)
(linked to in the alert "Tip" callout on the
[main NQC GitHub page](https://github.com/BrickBot/nqc)).

This reverts commit 6d6b64b.
@lmoellendorf
Copy link
Contributor Author

Ah, sorry. I just noticed that there are 2 Getting started sections now. Please do not merge yet, I will come up with another solution. Also I found some minor typing errors in my installation instructions.

@lmoellendorf lmoellendorf marked this pull request as draft July 31, 2024 18:05
@mesheets
Copy link
Member

Absolutely—no worries! Also, please don't feel obligated to retain the "(Original) NQC ReadMe" section in README.md as-is. It was basically just left in there because there wasn't anything better at the time with which to update or replace it.

@lmoellendorf lmoellendorf marked this pull request as ready for review August 1, 2024 09:06
@mesheets mesheets merged commit fc79867 into BrickBot:master Feb 16, 2025
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