Bug 7366 - NodeJS is needed to build and test noVNC
Summary: NodeJS is needed to build and test noVNC
Status: CLOSED FIXED
Alias: None
Product: ThinLinc
Classification: Unclassified
Component: Build system (show other bugs)
Version: trunk
Hardware: PC Unknown
: P2 Normal
Target Milestone: 4.11.0
Assignee: Pierre Ossman
URL:
Keywords: prosaic, samuel_tester
Depends on: 7362 7369
Blocks: 7367 7368
  Show dependency treegraph
 
Reported: 2019-08-15 11:00 CEST by Pierre Ossman
Modified: 2019-11-12 15:48 CET (History)
1 user (show)

See Also:
Acceptance Criteria:


Attachments

Description Pierre Ossman cendio 2019-08-15 11:00:41 CEST
noVNC uses NodeJS in a lot of development scripts and for the unit tests. We'd like to be able to use these as well, so we need to get the requirements in to our build environment.

What we need is NodeJS itself, and an easy way to get NPM packages in to the build system (at least for things that will be automated).
Comment 1 Pierre Ossman cendio 2019-08-15 11:13:43 CEST
Unfortunately NodeJS triggers a bug in our gcc, so we need to upgrade that. It also requires OpenSSL 1.1+, so we also have to fix that.
Comment 5 Pierre Ossman cendio 2019-10-24 17:08:31 CEST
Packaging done now. Tested by producing all the packaging for eslint and checking that it works.

Actually packaging some nodejs modules will be done on the bugs that require specific modules.
Comment 6 Samuel Mannehed cendio 2019-10-31 15:45:57 CET
Works great. Verified by doing the following:

1. Made a test program that required and used the nodejs package called 'mocha'
2. Verified that running the test program in cbrun didn't work
3. Created RPMs for mocha and all it's dependencies
4. Installed all the new RPMs
5. Verified that the test program now worked

Note You need to log in before you can comment on or make changes to this bug.