[ https://issues.apache.org/jira/browse/CB-13964?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=16400938#comment-16400938 ]
ASF GitHub Bot commented on CB-13964: ------------------------------------- dblotsky commented on a change in pull request #806: CB-13964: Fixed make clean for windows URL: https://github.com/apache/cordova-docs/pull/806#discussion_r174890234 ########## File path: Makefile ########## @@ -287,6 +300,19 @@ $(CSS_DEST_DIR)/%.css: $(CSS_SRC_DIR)/%.css $(call printfile,$<) >> $@ # maintenance +ifdef WINDOWS +clean: + $(RM) $(VERSION_CONFIG) + $(RM) $(DEFAULTS_CONFIG) + $(RM) $(DOCS_PAGE_LIST) + $(RM) $(DOCS_VERSION_DATA) + $(RM) $(TOC_FILES) + $(RM) $(PLUGINS_APP) + $(RM) $(FETCHED_FILES) + IF EXIST "$(CSS_DEST_DIR)" $(RMDIR) $(CSS_DEST_DIR) + IF EXIST "$(PROD_DIR)" $(RMDIR) $(PROD_DIR) + IF EXIST "$(DEV_DIR)" $(RMDIR) $(DEV_DIR) Review comment: As above, you can avoid the `if`/`else` here by defining `RM` differently. Also, you can avoid the `IF EXIST` by adding a `-` to the beginning of the line, which will tell Make to ignore any errors. So: ``` -$(RMDIR) $(CSS_DEST_DIR) ``` instead of ``` $(RMDIR) $(CSS_DEST_DIR) ``` ---------------------------------------------------------------- This is an automated message from the Apache Git Service. To respond to the message, please log on GitHub and use the URL above to go to the specific comment. For queries about this service, please contact Infrastructure at: us...@infra.apache.org > 'Make' command to build cordova website dont work in windows > ------------------------------------------------------------ > > Key: CB-13964 > URL: https://issues.apache.org/jira/browse/CB-13964 > Project: Apache Cordova > Issue Type: Bug > Components: cordova-docs > Reporter: Gandhirajan > Priority: Critical > > Make commands for cordova website devleopment dont work in windows OS -- This message was sent by Atlassian JIRA (v7.6.3#76005) --------------------------------------------------------------------- To unsubscribe, e-mail: issues-unsubscr...@cordova.apache.org For additional commands, e-mail: issues-h...@cordova.apache.org