Less.js: नोड v6.0.0 . पर टूटा हुआ

कम स्वयं किसी भी ObjectTemplate.Set फ़ंक्शन का उपयोग नहीं करता है, इसलिए यह शायद इसकी निर्भरता में से एक है। वर्तमान मास्टर में निर्भरता अद्यतनों की लंबी श्रृंखला है इसलिए अभी यह बताना कठिन है कि v6 -संगत रिलीज़ प्राप्त करना कितना कठिन हो सकता है। बेझिझक इसकी गहराई से जांच करें और एक पैच/पीआर सुझाएं।

यह नोडमॉन के साथ एक समस्या की तरह दिखता है, कम नहीं

वही देख रहे हैं। किसी भी तुच्छ .less फ़ाइल पर लेससी चलाने से कोई बिल्ड फ़ाइल नहीं निकलती है और Node.js v6 के तहत कोई त्रुटि नहीं होती है, यहां तक ​​कि वर्बोज़ मोड में भी।

मुझे इसका कारण मिल गया है:

कम कॉल का वर्तमान प्रकाशित संस्करण path.dirname() के साथ undefined जो नोड v6 के साथ एक त्रुटि फेंकता है। इसे पहले ही ec04a03f1cba3a092d5cd7f7c5d8e28bb43c1932 द्वारा संबोधित किया जा चुका है लेकिन अभी तक प्रकाशित नहीं किया गया है। त्रुटि को फेंका नहीं गया है क्योंकि lessc का वर्तमान कार्यान्वयन कम के कारण होने वाली सभी त्रुटियों को निगल जाता है। मैंने इसे ठीक करने के लिए एक पीआर बनाया है।

तो, एम्म, किसका विलय करना है? #2882 या #2884? :)

वे दो अलग-अलग मुद्दों को ठीक करते हैं IMO

तो दोनों? (मैं पूछ रहा हूं क्योंकि वे कुछ हद तक ओवरलैप करते हैं और दोनों को विलय करने के लिए कुछ अतिरिक्त संपादन की आवश्यकता होगी)।

दूसरा सीधे 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

इसे स्पष्ट करने के लिए Thx। मुझे नहीं पता था कि process.exit() नॉन-ब्लॉकिंग है। मैं अपने पीआर को यथाशीघ्र अपडेट करने जा रहा हूं।

हालांकि एक बात: अगर process.exit() फ्लश नहीं किया तो process.stdout - मेरे कंसोल में आउटपुट क्यों है? मुस्कराहट:

यह संभव है कि स्ट्रीम बंद होने तक पहले ही फ्लश हो गया हो। यह सिर्फ गारंटी नहीं है। इसलिए कंसोल में कभी-कभी (आंशिक) आउटपुट होता है।

पुष्टि की कि, कम से कम हमारे लिए, सिर में चल रहा है (जिसमें https://github.com/less/less.js/commit/ec04a03f1cba3a092d5cd7f7c5d8e28bb43c1932 शामिल है), यह हल हो जाता है।

"--source-map" का उपयोग करना अभी भी मेरे लिए एक खाली सीएसएस फ़ाइल उत्पन्न करता है .. (अन्यथा मेरे लिए काम कर रहा है)। node.js v6.0.0 विंडोज़ 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)

मुझे एक पीआर खोलना ठीक रहेगा जो इसे ठीक करता है, लेकिन मुझे यकीन नहीं है कि किन स्थितियों में स्रोत मानचित्र विकल्प समर्थित हैं, उदाहरण के लिए जब आउटपुट process.stdout है। यह कोड थोड़ा जटिल है और शायद कुछ रिफैक्टरिंग की आवश्यकता है ...

पुष्टि करें, यह नोड 6 के साथ टूट जाता है। कोई त्रुटि आउटपुट नहीं है :(

क्या हम अभी भी इसे ठीक करने के लिए किसी जनसंपर्क की प्रतीक्षा कर रहे हैं?

AFAICT, यह सिर पर तय है, इसलिए npm पैकेज प्रकाशित होने की प्रतीक्षा कर रहा है।

(दूसरों को पुष्टि करनी चाहिए।)

@lmeyerov मैं इसे कमिट में नहीं देखता। और पीआर #2882 या #2884 जुड़े हुए हैं, लेकिन कोई भी यह नहीं कहता है कि यह नोड 6 के लिए एक फिक्स है। तो, मेरी जानकारी के लिए, एक पीआर अभी तक मौजूद नहीं है, भले ही @jhnns को इसका कारण मिल गया हो? या, क्या उन पीआर में से एक नोड 6 संगतता को ठीक करता है लेकिन अन्य बग (स्रोत मानचित्र) पेश करता है जिन्हें अभी तक ठीक नहीं किया गया है? क्या कोई इसे स्पष्ट कर सकता है?

@ मैथ्यू-डीन जिस मुद्दे को हमने मारा, वह lessc के रूप में सामने आया, कुछ भी उत्सर्जित नहीं कर रहा था, मैं _थिंक_ इसे हल किया गया था। https://github.com/less/less.js/commit/ec04a03f1cba3a092d5cd7f7c5d8e28bb43c1932 । एनपीएम पर वर्तमान में प्रकाशित संस्करण और 22 अप्रैल की मेनलाइन के बीच कुछ ने इसे हमारे लिए हल कर दिया।

@ मैथ्यू-डीन ec04a03f1cba3a092d5cd7f7c5d8e28bb43c193 का उपयोग कर मेरे लिए काम करता है (मैंने इसे यहां कहीं देखा)।

@lmeyerov @matthew-dean _source map_ अभी भी मेरे लिए, दोनों सिरों पर और https://github.com/less/less.js/commit/ec04a03f1cba3a092d5cd7f7c5d8e28bb43c1932 =/

बीटीडब्ल्यू, त्रुटि रिपोर्टिंग बगफिक्स के बिना भी, मेरे ec04a03 पैच वाले संस्करण को जारी करने के खिलाफ बोलने वाला कुछ भी नहीं होना चाहिए, ताकि कम से कम Node.js v6 पर पहले की तरह काम करे।

@addaleax कम 2.7.0 जारी किया गया है (अभी-अभी)। कौन से बग अभी भी बकाया हैं? मुझे इसे इस रिलीज़ में "ज्ञात बग" के रूप में जोड़ना चाहिए।

केवल यही एक _I_ का सामना करना पड़ा है, साथ ही विफलता के मामले में कोई वास्तविक त्रुटि आउटपुट नहीं होने का "बग" है। हालांकि दूसरों के लिए बोल नहीं सकते।

किसी कारण से, स्रोत मानचित्र सभी कम परीक्षण पास करते हैं, लेकिन अपग्रेड करने के बाद, मुझे स्रोत मानचित्र त्रुटि भी मिल रही है, न कि नोड 6.0 वातावरण में। (नोड 4 में हो रहा है) तो कुछ बदलाव है जिसने अभी-अभी सोर्स मैप्स को तोड़ा है।

मेरा मानना ​​है कि पीआर #2834 ने सोर्स मैप बग पेश किया। मेरे सिस्टम पर 470af20 कमिट करने से मेरा सोर्स मैप इश्यू ठीक हो गया। क्या कोई (@ewebdev) तुरंत पुष्टि कर सकता है? यदि ऐसा है, तो मैं प्रत्यावर्तन करूंगा और 2.7.1 हॉटफिक्स डालूंगा।

//cc @nicks

@ मैथ्यू-डीन रिवर्टिंग कमिट https://github.com/less/less.js/commit/470af20f641c9b5389475733dfe341aa4c180fba मेरे एनवी पर सोर्स मैप्स की समस्या को हल नहीं करता है ।

@ मैथ्यू-डीन क्षमा करें! यह सिर्फ एक प्रदर्शन अनुकूलन है, अगर यह समस्याएं पैदा कर रहा है तो वापस करने के लिए स्वतंत्र महसूस करें और हम बाद में फिर से प्रयास कर सकते हैं

इतना भ्रम...

चूंकि मैंने पहले ही इसमें जांच के लिए कुछ समय दिया है, इसलिए मैं इसे स्पष्ट कर दूं:

• ec04a03f1cba3a092d5cd7f7c5d8e28bb43c1932 bin/lessc में एक बग को ठीक करता है जहां एक अपरिभाषित पथ dirname को पास किया गया था। यह अब प्रकाशित हो गया है, सब कुछ ठीक है
• bin/lessc में अभी भी एक बग है, जहां एक अपरिभाषित पथ basename को पास किया गया है । बस lessc --source-map-map-inline some-file.less चलाएं और कम से स्टडआउट के लिए कुछ भी नहीं निकलेगा। इस बग के लिए अभी तक कोई पीआर नहीं है। मैंने इसे ठीक करने का प्रयास किया है, लेकिन चूंकि lessc के लिए कोई परीक्षण नहीं हैं और बहुत सारे अलग-अलग विकल्प हैं, इसलिए मुझे विश्वास नहीं था कि मैं कुछ और नहीं तोड़ सकता...
• उपयोगकर्ता को दोनों बग की सूचना नहीं दी गई थी (उदाहरण के लिए स्टैक ट्रेस या गैर-शून्य निकास कोड प्रस्तुत करना)। यह मेरे पीआर द्वारा तय किया गया है जो अभी भी लंबित है।

@jhnns ओह, अच्छा, मैंने उस स्रोत मानचित्र को नहीं देखा।

यदि और कुछ नहीं, तो कोई भी path.basename(output) को path.basename(output || 'undefined'); में बदलकर पुराने नोड v5 व्यवहार को _restore_ कर सकता है ... मुझे लगता है कि वहां के पूरे ब्लॉक को output === undefined के लिए छोड़ दिया जा सकता है।

@jhnns bin/lessc में बग का क्या प्रभाव है। क्या यह एक महत्वपूर्ण बग है? क्या कम अभी भी पूरा होता है? इसका परिणाम क्या है? क्या आप अधिक विवरण के साथ इस बग के लिए कोई विशिष्ट समस्या दर्ज कर सकते हैं?

मैंने आपके पीआर को त्रुटि रिपोर्टिंग के लिए मर्ज कर दिया है।

@ मैथ्यू-डीन देखें https://github.com/less/less.js/issues/2896

[संपादित करें: कोई बात नहीं, धन्यवाद @ मैथ्यू-डीन]

@ewebdev यदि आपको अभी भी कम 2.7.1 पर कोई समस्या आ रही है, तो क्या आप अधिक विवरण के साथ एक अलग समस्या दर्ज कर सकते हैं?

@claar यह एक अलग मुद्दा है ( @jhnns की सूची में पहला), जिसे संबोधित किया गया है। दूसरा आइटम एक अलग बग है, और @ewebdev एक अलग बग है। स्पष्टता के लिए उन्हें इस मुद्दे से अलग करने की जरूरत है।

@ निक्स ऐसा होता है। आप पहले नहीं होंगे। यदि आप परीक्षण जोड़ने में हमारी सहायता कर सकते हैं ताकि हम भविष्य में काम नहीं कर रहे स्रोत मानचित्रों का पता लगा सकें, तो इसकी बहुत सराहना की जाएगी।

बीटीडब्ल्यू, एक सामान्य एफवाईआई के रूप में, लेस का देव समर्थन अभी थोड़ा पतला है (हमारे प्रमुख योगदानकर्ताओं में से एक को पारिवारिक जिम्मेदारियों के कारण पीछे हटना पड़ा), इसलिए हम निश्चित रूप से कम डेवलपर्स का संगठनात्मक स्तर पर शामिल होने का स्वागत करेंगे। यदि आपके कोई प्रश्न हैं तो आप मुझसे अलग से संपर्क कर सकते हैं।

मैंने एक अलग मुद्दा बनाया है। चूंकि मूल समस्या का समाधान कर दिया गया है, इसलिए हम इसे बंद कर सकते हैं।

@jhnns अच्छा लगता है। फॉलो अप करने के लिए धन्यवाद।

अभी भी नोड v6.9.2 LTS पर टूटा हुआ है।
कम चलने पर कुछ नहीं होता