Re: JS Test error on Windows
| От | Aditya Toshniwal |
|---|---|
| Тема | Re: JS Test error on Windows |
| Дата | |
| Msg-id | CAM9w-_=Apw6tLwjwf2EmuuXGfwkp5FMp12S-6FJVgO6bzhwvAQ@mail.gmail.com обсуждение исходный текст |
| Ответ на | Re: JS Test error on Windows (Dave Page <dpage@pgadmin.org>) |
| Ответы |
Re: JS Test error on Windows
|
| Список | pgadmin-hackers |
Hi Dave,
The above error was not thrown when import 'jquery.event.drag' was added to column_selector_
So the doubt was on webpack as it was not importing jquery.event.drag even though it was added using imports-loader, the below test:
test: /.*slickgrid\/+slick\.(?!core)*/,
loader: 'imports-loader?' +
'jquery.ui' +
',jquery.event.drag' +
',slickgrid',
I noticed later that the test regex is not just for files but also included path. So, gave try by replacing "\/" with "\\" and it worked.
But then this will not work on Unix, and so changed it to [\\\/]+ taking both styles.
Thanks and Regards,
Aditya Toshniwal
Software Engineer | EnterpriseDB Software Solutions | Pune
"Don't Complain about Heat, Plant a tree"
On Fri, May 11, 2018 at 3:05 PM, Dave Page <dpage@pgadmin.org> wrote:
On Fri, May 11, 2018 at 7:28 AM, Aditya Toshniwal <aditya.toshniwal@--enterprisedb.com> wrote: Hi Dave/Hackers,PFA fix for the problem. The RegEx for webpack test is not working on windows because the path separator is different for Windows( \ ) and Unix( / ). And so, the imports loader is not executing for slickgrid.Patch tested on Win 10, Win 7, Ubuntu, Cent OS and works fine.Nice catch! How did you figure that one out?Patch applied.Dave Page
Blog: http://pgsnake.blogspot.com
Twitter: @pgsnake
EnterpriseDB UK: http://www.enterprisedb.com
The Enterprise PostgreSQL Company
В списке pgadmin-hackers по дате отправления: