Less.js: 노드 v6.0.0에서 손상됨

Less 자체는 ObjectTemplate.Set 함수를 사용하지 않으므로 종속성 중 하나일 수 있습니다. 현재 마스터에는 긴 종속성 업데이트 체인이 있으므로 v6 호환 릴리스를 얻는 것이 얼마나 어려운지 아직 말하기 어렵습니다. 스스로 더 깊이 조사하고 패치/PR을 제안하십시오.

그것은 nodemon의 문제처럼 보입니다.

같은 것을보고. 사소한 .less 파일에서 lessc를 실행하면 상세 모드에서도 Node.js v6에서 빌드 파일과 오류가 출력되지 않습니다.

원인을 찾았습니다.

현재 게시된 Less 버전은 undefined path.dirname() 를 호출하여 노드 v6에서 오류를 발생시킵니다. 이것은 이미 ec04a03f1cba3a092d5cd7f7c5d8e28bb43c1932에 의해 해결되었지만 아직 게시되지 않았습니다. lessc 의 현재 구현이 Less 자체로 인해 발생하는 모든 오류를 삼키기 때문에 오류가 발생하지 않습니다. 이 문제를 해결하기 위해 PR을 만들었습니다.

자, 어느 쪽을 병합할까요? #2882 또는 #2884? :)

그들은 두 가지 다른 문제를 해결합니다.

그래서 둘 다? (둘 다 약간 겹치고 병합하려면 추가 편집이 필요하기 때문에 묻습니다.)

다른 하나는 process.exit() 를 직접 사용합니다. 이는 stdout/stderr이 플러시되지 않도록 방지해야 합니다.

@evanlucas process.exit() 를 직접 호출할 때 발생하는 이 문제를 어떻게 재현할 수 있는지 설명해 주시겠습니까? process.exit() 는 드문 경우에 사용해야 한다는 데 동의합니다. 일반적으로 정기적으로 종료하거나 오류를 발생시켜 종료하는 것이 더 낫기 때문입니다. 그러나 이 경우 예상되는 오류에 대한 스택 추적을 표시하고 싶지 않기 때문에 오류 코드가 있는 process.exit() 가 더 적절하다고 생각합니다. 그리고 실행을 계속하지만 플래그로 추가 처리를 방지하는 것도 이상하게 보입니다(현재 구현된 것처럼).

process.exitCode 설정은 스택 추적을 표시하지 않습니다. process.exit() 는 실제로 process.stdout 또는 process.stderr 를 플러시하지 않습니다(둘 다 차단되지 않음). 드문 경우에만 명시적으로 호출해야 합니다. 그렇지 않으면 모든 핸들을 닫으면 이벤트 루프가 저절로 닫혀 프로세스가 종료됩니다.

관련: https://github.com/nodejs/node/issues/6409

이를 명확히 해 주셔서 감사합니다. process.exit() 가 비차단인지 몰랐습니다. 최대한 빨리 PR을 업데이트하겠습니다.

하지만 한 가지: process.exit() 가 process.stdout 를 플러시하지 않은 경우 – 내 콘솔에 출력이 있는 이유는 무엇입니까? :이를 드러내고 웃다:

스트림이 닫힐 때 이미 플러시되었을 수 있습니다. 그것은 단지 보증이 아닙니다. 이것이 콘솔에 때때로 (부분) 출력이 있는 이유입니다.

적어도 우리의 경우 헤드(https://github.com/less/less.js/commit/ec04a03f1cba3a092d5cd7f7c5d8e28bb43c1932 포함)에서 실행되면 이것이 해결된다는 것을 확인했습니다.

"--source-map"을 사용하면 여전히 빈 CSS 파일이 생성됩니다. (그렇지 않으면 저에게 효과적입니다). node.js v6.0.0(Windows 8 64비트)

@ewebdev 네, 확인할 수 있습니다. #2891을 사용하면 다음이 발생합니다.

lessc --source-map-map-inline styles/main.less
path.js:7
    throw new TypeError('Path must be a string. Received ' + inspect(path));
    ^

TypeError: Path must be a string. Received undefined
    at assertPath (path.js:7:11)
    at Object.basename (path.js:1355:5)
    at /Users/jhnns/dev/jhnns/less.js/bin/lessc:292:61
    at Object.<anonymous> (/Users/jhnns/dev/jhnns/less.js/bin/lessc:486:3)
    at Module._compile (module.js:541:32)
    at Object.Module._extensions..js (module.js:550:10)
    at Module.load (module.js:456:32)
    at tryModuleLoad (module.js:415:12)
    at Function.Module._load (module.js:407:3)
    at Function.Module.runMain (module.js:575:10)

이 문제를 해결하는 PR을 열어도 되지만 출력이 process.stdout 인 경우와 같이 어떤 상황에서 어떤 소스 맵 옵션이 지원되는지 잘 모르겠습니다. 이 코드는 약간 복잡하고 아마도 약간의 리팩토링이 필요할 것입니다...

확인, 이것은 노드 6과 중단됩니다. 오류가 출력되지 않습니다 :(

이 문제를 해결하기 위해 여전히 PR을 기다리고 있습니까?

AFAICT, 이것은 헤드에 고정되어 있으므로 npm 패키지 게시를 기다립니다.

(다른 사람들은 확인해야 합니다.)

@lmeyerov 커밋에서 볼 수 없습니다. 그리고 PR #2882 또는 #2884가 연결되어 있지만 아무도 이것이 Node 6에 대한 수정이라고 말하지 않습니다. 그래서 내가 알기로는 @jhnns 가 원인을 찾았는데도 아직 PR이 존재하지 않는 건가요? 또는 해당 PR 중 하나가 노드 6 호환성을 수정하지만 아직 수정되지 않은 다른 버그(소스 맵)를 도입합니까? 누군가 이것을 명확히 할 수 있습니까?

@matthew-dean lessc 아무 것도 방출하지 않는 것으로 표면화된 문제의 경우 _생각_ 하여 해결되었습니다. https://github.com/less/less.js/commit/ec04a03f1cba3a092d5cd7f7c5d8e28bb43c1932 . 현재 npm에 게시된 버전과 4월 22일 메인라인 사이에 문제가 해결되었습니다.

ec04a03f1cba3a092d5cd7f7c5d8e28bb43c193을 사용하는 @matthew-dean이 저에게 효과적입니다.

@lmeyerov @matthew-dean _source map_ 머리와 https://github.com/less/less.js/commit/ec04a03f1cba3a092d5cd7f7c5d8e28bb43c1932 =/

Btw, 오류 보고 버그 수정이 없더라도 내 ec04a03 패치가 포함된 버전을 출시하는 것에 반대하는 것은 없어야 합니다. 그래야 lessc가 적어도 이전과 같이 Node.js v6에서 작동합니다.

@addaleax Less 2.7.0이 출시되었습니다(지금 막). 아직 해결되지 않은 버그는 무엇입니까? 이 릴리스에 "알려진 버그"로 추가해야 합니다.

그것은 _I_가 마주한 유일한 것입니다. 게다가 실패 시 실제 오류 출력이 없는 "버그"도 있습니다. 그러나 다른 사람을 위해 말할 수 없습니다.

어떤 이유에서인지 소스 맵은 모든 Less 테스트를 통과했지만 업그레이드 후 Node 6.0 환경이 아닌 소스 맵 오류도 발생합니다. (노드 4에서 발생) 그래서 소스 맵을 깨뜨린 몇 가지 변경 사항이 있습니다.

PR #2834가 소스 맵 버그를 도입했다고 생각합니다. 내 시스템에서 커밋 470af20을 되돌리면 소스 맵 문제가 해결되었습니다. 누군가(@ewebdev)가 빠르게 확인할 수 있습니까? 그렇다면 되돌리기를 커밋하고 2.7.1 핫픽스를 내놓을 것입니다.

//cc @닉네임

@matthew-dean 커밋 되돌리기 https://github.com/less/less.js/commit/470af20f641c9b5389475733dfe341aa4c180fba 내 환경에서 소스 맵 문제를 해결 하지 않습니다 .

@matthew-dean 죄송합니다! 성능 최적화일 뿐입니다. 문제가 발생하면 언제든지 되돌릴 수 있으며 나중에 다시 시도할 수 있습니다.

너무 혼란스러워... 😁

조사를 위해 이미 시간을 들였기 때문에 명확히 해두겠습니다.

• ec04a03f1cba3a092d5cd7f7c5d8e28bb43c1932는 정의되지 않은 경로가 dirname 로 전달된 bin/lessc 의 버그를 수정합니다. 이것은 이제 게시되었습니다, 모든 것이 좋습니다 👍
• bin/lessc 에는 여전히 버그가 있습니다. 여기서 정의되지 않은 경로는 basename 로 전달됩니다 . lessc --source-map-map-inline some-file.less 를 실행하면 less는 stdout에 아무 것도 내보내지 않습니다. 이 버그에 대한 PR은 아직 없습니다. 이거 고치려고 했는데 lessc 에 대한 테스트도 없고 옵션도 너무 다양해서 다른건 깨지지 않을만큼 자신이 없었습니다... 😞
• 두 버그 모두 사용자에게 보고되지 않았습니다(예: 스택 추적 또는 0이 아닌 종료 코드 표시). 이것은 아직 보류 중인 내 PR 에 의해 수정되었습니다.

@jhnns 오, 멋지네요. 저는 그 소스 맵을 보지 못했습니다.

다른 방법이 없다면 path.basename(output) 를 path.basename(output || 'undefined'); 로 변경하여 이전 Node v5 동작을 _복원__할 수 있습니다. 하지만 output === undefined 에 대해 전체 블록을 건너뛸 수 있습니다.

@jhnns bin/lessc 버그의 영향은 무엇입니까? 크리티컬 버그인가요? Less는 여전히 완료됩니까? 결과는 무엇입니까? 이 버그에 대한 특정 문제를 더 자세한 정보와 함께 신고할 수 있습니까?

오류 보고를 위해 귀하의 PR을 병합했습니다.

@matthew-dean 참조 https://github.com/less/less.js/issues/2896

[편집: 신경 쓰지 마세요, 감사합니다 @matthew-dean]

@ewebdev Less 2.7.1에서 여전히 문제가 있는 경우 자세한 내용을 포함하여 별도의 문제를 제출할 수 있습니까?

@claar 그것은 해결된 별도의 문제( @jhnns 목록의 첫 번째 문제)입니다. 두 번째 항목은 별도의 버그이고 @ewebdev 는 별도의 버그입니다. 명확성을 위해 이 문제와 분리해야 합니다.

@nicks 그것은 발생합니다. 당신이 첫 번째는 아닐 것입니다. 앞으로 작동하지 않는 소스 맵을 감지할 수 있도록 테스트를 추가하는 데 도움을 주시면 감사하겠습니다.

Btw, 일반적인 참고로, Less의 개발 지원은 현재 약간 희박합니다(우리의 주요 기여자 중 한 명이 가족 책임 때문에 물러나야 했습니다). 따라서 우리는 Less 개발자가 조직 수준에서 참여하는 것을 확실히 환영할 것입니다. 질문이 있는 경우 저에게 별도로 연락할 수 있습니다.

별도의 문제를 만들었습니다. 원래 문제가 해결되었으므로 이 문제를 닫을 수 있습니다.

@jhnns 좋은 것 같습니다. 팔로우해주셔서 감사합니다.

노드 v6.9.2 LTS에서 여전히 중단되었습니다.
lessc 를 실행할 때 아무 일도 일어나지 않습니다.