neilcsmith-net commented on code in PR #4205:
URL: https://github.com/apache/netbeans/pull/4205#discussion_r902691182
##########
java/java.disco/src/org/netbeans/modules/java/disco/QuickPkgSelection.java:
##########
@@ -57,6 +59,9 @@ public QuickPkgSelection(QuickPanel.QuickSelection quick) {
switch (OS.getOperatingSystem()) {
case WINDOWS:
return ArchiveType.MSI == p.getArchiveType() ||
ArchiveType.EXE == p.getArchiveType();
+ case LINUX:
Review Comment:
Finally getting a chance to look through these changes. This doesn't look
right? Should be returning things like DEB and RPM? However, how do we return
the right one of those anyway? I do wonder if "installers" should be left for
the advanced panel and we remove that option entirely here?
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: [email protected]
For queries about this service, please contact Infrastructure at:
[email protected]
---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]
For further information about the NetBeans mailing lists, visit:
https://cwiki.apache.org/confluence/display/NETBEANS/Mailing+lists
