[
https://issues.apache.org/jira/browse/CB-14156?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=16619298#comment-16619298
]
ASF subversion and git services commented on CB-14156:
------------------------------------------------------
Commit 0b56cbc27bf580a63d11a6d8ed9c6fdb205dd829 in cordova-js's branch
refs/heads/master from [~brodybits]
[ https://gitbox.apache.org/repos/asf?p=cordova-js.git;h=0b56cbc ]
Merge pull request #162 from brodybits/cb-remove-browserify
[CB-14156] Remove browserify
> cordova-js browserify issues
> ----------------------------
>
> Key: CB-14156
> URL: https://issues.apache.org/jira/browse/CB-14156
> Project: Apache Cordova
> Issue Type: Bug
> Components: cordova-js
> Reporter: Chris Brody
> Assignee: Chris Brody
> Priority: Major
>
> I spotted the following issues with the way {{cordova-js}} uses
> {{browserify}}:
> * 1: duplicate source files in {{src}}: {{src/cordova_b.js}},
> {{src/common/modulemapper_b.js}},
> {{src/common/pluginloader_b.js}}, {{src/common/init_b.js}}, which only seem
> to be used by {{tasks/lib/bundle-browserify.js}}
> * 2: I have not been able to make a working {{browserify}} build.
> * 3: {{npm outdated}} shows {{browserify}} as outdated
> Another possible issue that I am not 100% sure about:
> * 4: {{browserify}} is listed as a {{dependencies}} item in {{package.json}}
> while I think it should have been listed as a {{devDependencies}} item
> I am very concerned that keeping duplicated source files may lead to old bugs
> including possible security issues leaking through the maintenance process in
> the future. The changes between the duplicated source files and the
> non-browserify versions are very limited so it should be possible to resolve
> this issue using very basic code techniques.
> I think the easiest solution would be to drop the browserify build/test
> target and remove the duplicated source files. I suspect that the browserify
> target has not been used for years.
--
This message was sent by Atlassian JIRA
(v7.6.3#76005)
---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]
