EssamWisam / cmp-docs

A comprehensive guide for prospective, current and past students in the computer engineering department of Cairo university.
https://cmp-docs.pages.dev
52 stars 8 forks source link

Fix `npm start` for linux. #22

Closed KnockerPulsar closed 5 months ago

KnockerPulsar commented 8 months ago

closes #21

netlify[bot] commented 8 months ago

Deploy Preview for cmp-docs ready!

Name Link
Latest commit 8bda1752b9e92e0a2f9cd86cfa359b215f04fd67
Latest deploy log https://app.netlify.com/sites/cmp-docs/deploys/6541a6c2c8d74d0008c63b46
Deploy Preview https://deploy-preview-22--cmp-docs.netlify.app
Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify site configuration.

EssamWisam commented 8 months ago

This breaks it for me on Mac OS. Getting an error along the lines of

  opensslErrorStack: [ 'error:03000086:digital envelope routines::initialization error' ],
  library: 'digital envelope routines',
  reason: 'unsupported',
  code: 'ERR_OSSL_EVP_UNSUPPORTED'
}

You could follow @Iten-No-404's recommendation and make your command into start_lin if you are sure this would make it work on all Linux. I am surprised whatsoever because Mac usually behaves quite similarly in the shell.

It may help me later resolve this issue further if you inform me of your Node.js version and whether it works or not when start is just react-scripts start.

EssamWisam commented 5 months ago

This has been gathering dust for some time @KnockerPulsar should we close this or are you willing to make a separate entry for start_lin?

EssamWisam commented 5 months ago

Can be closed thanks to #42