Skip to contentSkip to conversation

bootstrap: remove development/debug files

PATCH New Yegor Yefremov 2 participants 2 messages Jan 12, 2015

lore
Share & tools
lore

Yegor Yefremov

author
#0 lore raw

PATCH 1/1 bootstrap: remove development/debug files

*.min.* files are sufficient for a production system, so remove other files, that will be used during development stage.

Signed-off-by Yegor Yefremov <[email protected]>
---
 package/bootstrap/bootstrap.mk | 5 +++++
 1 file changed, 5 insertions(+)
package/bootstrap/bootstrap.mk +5 -0
index 1bf8aa2..f79a00f 100644
@@ -16,6 +16,11 @@ endef
 define BOOTSTRAP_INSTALL_TARGET_CMDS
 	mkdir -p $(TARGET_DIR)/var/www/bootstrap
 	cp -dpfr $(@D)/dist/* $(TARGET_DIR)/var/www/bootstrap
+
+	rm $(TARGET_DIR)/var/www/bootstrap/css/bootstrap-theme.css*
+	rm $(TARGET_DIR)/var/www/bootstrap/css/bootstrap.css*
+	rm $(TARGET_DIR)/var/www/bootstrap/js/npm.js
+	rm $(TARGET_DIR)/var/www/bootstrap/js/bootstrap.js
 endef
 
 $(eval $(generic-package))
Signature · 1 line
2.1.0
#1 lore raw

PATCH 1/1 bootstrap: remove development/debug files

Dear Yegor Yefremov,

Quoted 15 lines · Yegor Yefremov

*.min.* files are sufficient for a production system, so remove other files, that will be used during development stage.

Signed-off-by Yegor Yefremov <[email protected]>
---
 package/bootstrap/bootstrap.mk | 5 +++++
 1 file changed, 5 insertions(+)
package/bootstrap/bootstrap.mk +0 -0
index 1bf8aa2..f79a00f 100644
@@ -16,6 +16,11 @@ endef
 define BOOTSTRAP_INSTALL_TARGET_CMDS

package/bootstrap/bootstrap.mk endef · line 26

Quoted 10 lines
package/bootstrap/bootstrap.mk +5 -0
 	mkdir -p $(TARGET_DIR)/var/www/bootstrap
 	cp -dpfr $(@D)/dist/* $(TARGET_DIR)/var/www/bootstrap
+
+	rm $(TARGET_DIR)/var/www/bootstrap/css/bootstrap-theme.css*
+	rm $(TARGET_DIR)/var/www/bootstrap/css/bootstrap.css*
+	rm $(TARGET_DIR)/var/www/bootstrap/js/npm.js
+	rm $(TARGET_DIR)/var/www/bootstrap/js/bootstrap.js
 endef
 
 $(eval $(generic-package))

At this point, I am wondering if it is not simpler/shorter to install only the necessary files, rather than install everything and remove what's unneeded. Can you compare the two options?

Thanks,

Thomas

Signature · 3 lines
Thomas Petazzoni, CTO, Free Electrons
Embedded Linux, Kernel and Android engineering
http://free-electrons.com