[
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
