[Windows] Building fails for generated che-theia (eclipse-che/theia-generator)
See original GitHub issueTo build my own eclipse-che I followed the steps on https://www.npmjs.com/package/@eclipse-che/theia-generator):
git clone https://github.com/eclipse-theia/theia
cd theia
che-theia init
yarn
Note that I am running this on Windows 10, node version v12.16.3 and yarn version 1.22.4. Should this work?
The yarn operation failed, with the following error for a number of files:
lerna ERR! C:\work\theia\packages\@che-eclipse-che-theia-terminal\src\browser\server-definition\remote-connection.ts
lerna ERR! 1:1 error missing file header (tslint:file-header) @typescript-eslint/tslint/config
lerna ERR!
lerna ERR! C:\work\theia\packages\@che-eclipse-che-theia-terminal\src\browser\server-definition\remote-terminal-protocol.ts
lerna ERR! 1:1 error missing file header (tslint:file-header) @typescript-eslint/tslint/config
lerna ERR!
lerna ERR! C:\work\theia\packages\@che-eclipse-che-theia-terminal\src\browser\server-definition\terminal-proxy-creator.ts
lerna ERR! 1:1 error missing file header (tslint:file-header) @typescript-eslint/tslint/config
...
I did some debugging and found the error seems to be related to the License in the file header:
* SPDX-License-Identifier: EPL-2.0
If I change this into
* SPDX-License-Identifier: EPL-2.0 OR GPL-2.0 WITH Classpath-exception-2.0
it does not longer complain. (I found the license-identifier in other src files). I found the following lint rule theia\che\che-theia\generator\tslint.json, but don’t understand why it complain
{
"defaultSeverity": "error",
"rules": {
"file-header": [
true,
"SPDX-License-Identifier: EPL-2\\.0"
],
....
Any suggestion is welcome. Thanks!
Issue Analytics
- State:
- Created 3 years ago
- Comments:5 (3 by maintainers)
Top Results From Across the Web
Quay Container Registry
Hosted private Docker repositories. Includes full user management and history. Free for public repositories.
Read more >Top Related Medium Post
No results found
Top Related StackOverflow Question
No results found
Troubleshoot Live Code
Lightrun enables developers to add logs, metrics and snapshots to live code - no restarts or redeploys required.
Start FreeTop Related Reddit Thread
No results found
Top Related Hackernoon Post
No results found
Top Related Tweet
No results found
Top Related Dev.to Post
No results found
Top Related Hashnode Post
No results found
Top GitHub Comments
@hansbank Hello!
Theia
bumpedeslint
related versions to 3.1.0 For 3.0.0 versionclass-name-casing
was replaced bynaming-convention
: https://github.com/typescript-eslint/typescript-eslint/releases/tag/v3.0.0As @tsmaeder mentioned:
I’m working on it at the moment to align
che-theia
master withtheia
master. As work-around you could try some fixes from https://github.com/eclipse/che-theia/pull/823. For example, errors like:should be fixed by https://github.com/eclipse/che-theia/pull/823/commits/7e49ec77960e73e42591caafdbb94449c378244c The alignment with
theia
master is still in progress. I’ll try to finalize it ASAP.Issues go stale after
180
days of inactivity.lifecycle/stale
issues rot after an additional7
days of inactivity and eventually close.Mark the issue as fresh with
/remove-lifecycle stale
in a new comment.If this issue is safe to close now please do so.
Moderators: Add
lifecycle/frozen
label to avoid stale mode.