pwa-builder / PWABuilder-CLI

Node.js tool for App Generation
Other
1.63k stars 142 forks source link

Can't build Windows platform #212

Closed zackhall closed 8 years ago

zackhall commented 8 years ago

Running the following command:

 manifoldjs http://localhost:3000/ -d .\tmp -l debug -p windows -b

I see the following output:

[info ] manifoldjs  : Scanning http://localhost:3000/ for manifest...
[info ] manifoldjs  : Found a w3c manifest...
[debug] manifoldjs  : Manifest contents:
                      {
                          "lang": "en",
                          "name": "Windows HWA Starter Pack",
                          "short_name": "StarterPack",
                          "icons": [
                              {
                                  "src": "img/icons/Icon@16w.png",
                                  "sizes": "16x16",
                                  "type": "image/png"
                              },
                              {
                                  "src": "img/icons/Icon@32w.png",
                                  "sizes": "32x32",
                                  "type": "image/png"
                              },
                              {
                                  "src": "img/icons/Icon@48w.png",
                                  "sizes": "48x48",
                                  "type": "image/png"
                              },
                              {
                                  "src": "img/icons/Icon@54w.png",
                                  "sizes": "54x54",
                                  "type": "image/png"
                              },
                              {
                                  "src": "img/icons/Icon@64w.png",
                                  "sizes": "64x64",
                                  "type": "image/png"
                              },
                              {
                                  "src": "img/icons/Icon@90w.png",
                                  "sizes": "90x90",
                                  "type": "image/png"
                              },
                              {
                                  "src": "img/icons/Icon@106w.png",
                                  "sizes": "106x106",
                                  "type": "image/png"
                              },
                              {
                                  "src": "img/icons/Icon@120w.png",
                                  "sizes": "120x120",
                                  "type": "image/png"
                              },
                              {
                                  "src": "img/icons/Icon@128w.png",
                                  "sizes": "128x128",
                                  "type": "image/png"
                              },
                              {
                                  "src": "img/icons/Icon@170w.png",
                                  "sizes": "170x170",
                                  "type": "image/png"
                              },
                              {
                                  "src": "img/icons/Icon@256w.png",
                                  "sizes": "256x256",
                                  "type": "image/png"
                              },
                              {
                                  "src": "img/icons/Icon@270w.png",
                                  "sizes": "270x270",
                                  "type": "image/png"
                              },
                              {
                                  "src": "img/icons/Icon@360w.png",
                                  "sizes": "360x360",
                                  "type": "image/png"
                              },
                              {
                                  "src": "img/icons/Splash.png",
                                  "sizes": "1116x540",
                                  "type": "image/png"
                              }
                          ],
                          "scope": "/",
                          "start_url": "http://localhost:3000/index.html",
                          "display": "fullscreen",
                          "orientation": "landscape",
                          "theme_color": "blue",
                          "background_color": "blue"
                      }
[debug] manifoldjs  : Loading platform module: manifoldjs-cordova
[warn ] general     : Manifest validation SUGGESTION - It is recommended to specify a set of rules that represent the navigation scope of the application(member: mjs_extended_scope).
[warn ] general     : Manifest validation WARNING    - Unknown property (not in schema)(member: /lang).
[debug] manifoldjs  : Creating the 'Cordova Platform' app...
[info ] cordova     : Generating the Windows Platform app(s)...
[info ] cordova     : Creating the Cordova Platform project...
[debug] pid:20500   : Launching external process: 'C:\Users\zackhall\AppData\Roaming\npm\node_modules\manifoldjs\node_modules\.bin\cordova.cmd create cordova localhost.manifoldjs StarterPack'
[debug] pid:20500   : Creating a new cordova project.
[info ] cordova     : Copying the Cordova Platform manifest to the app folder...
[info ] cordova     : Adding the following plugins to the Cordova project: cordova-plugin-hostedwebapp@>=0.2.0 <0.3.0 cordova-plugin-whitelist@1.0.0...
[debug] pid:16900   : Launching external process: 'C:\Users\zackhall\AppData\Roaming\npm\node_modules\manifoldjs\node_modules\.bin\cordova.cmd plugin add cordova-plugin-hostedwebapp@>=0.2.0 <0.3.0 cordova-plugin-whitelist@1.0.0'
[debug] pid:16900   : (node:17436) fs: re-evaluating native module sources is not supported. If you are using the graceful-fs module, please update it to a more recent version.
[debug] pid:16900   : Fetching plugin "cordova-plugin-hostedwebapp@>=0.2.0 <0.3.0" via npm
[debug] pid:16900   : Fetching plugin "cordova-plugin-whitelist@1.0.0" via npm
[info ] cordova     : Adding the following Cordova platforms: windows...
[debug] pid:25204   : Launching external process: 'C:\Users\zackhall\AppData\Roaming\npm\node_modules\manifoldjs\node_modules\.bin\cordova.cmd platform add windows'
[debug] pid:25204   : Adding windows project...
[debug] pid:25204   : Running command: cmd "/s /c "C:\Users\zackhall\.cordova\lib\npm_cache\cordova-windows\4.1.0\package\bin\create.bat E:\github\hwa-starter\tmp\StarterPack\cordova\platforms\windows localhost.manifoldjs StarterPack""
[debug] pid:25204   : Creating Cordova Windows Project:
[debug] pid:25204   :   App Name  : StarterPack
[debug] pid:25204   :   Namespace : localhost.manifoldjs
[debug] pid:25204   :   Path      : E:\github\hwa-starter\tmp\StarterPack\cordova\platforms\windows
[debug] pid:25204   : Copying template to E:\github\hwa-starter\tmp\StarterPack\cordova\platforms\windows
[debug] pid:25204   : (node:17540) fs: re-evaluating native module sources is not supported. If you are using the graceful-fs module, please update it to a more recent version.
[debug] pid:25204   : Updating Cordova configuration from W3C manifest...
[debug] pid:25204   : Downloaded icon file: E:\github\hwa-starter\tmp\StarterPack\cordova\img\icons\Icon@48w.png
[debug] pid:25204   : Downloaded icon file: E:\github\hwa-starter\tmp\StarterPack\cordova\img\icons\Icon@54w.png
[debug] pid:25204   : Downloaded icon file: E:\github\hwa-starter\tmp\StarterPack\cordova\img\icons\Icon@64w.png
[debug] pid:25204   : Downloaded icon file: E:\github\hwa-starter\tmp\StarterPack\cordova\img\icons\Icon@32w.png
[debug] pid:25204   : Downloaded icon file: E:\github\hwa-starter\tmp\StarterPack\cordova\img\icons\Icon@16w.png
[debug] pid:25204   : Downloaded icon file: E:\github\hwa-starter\tmp\StarterPack\cordova\img\icons\Icon@106w.png
[debug] pid:25204   : Downloaded icon file: E:\github\hwa-starter\tmp\StarterPack\cordova\img\icons\Icon@120w.png
[debug] pid:25204   : Downloaded icon file: E:\github\hwa-starter\tmp\StarterPack\cordova\img\icons\Icon@90w.png
[debug] pid:25204   : Downloaded icon file: E:\github\hwa-starter\tmp\StarterPack\cordova\img\icons\Icon@128w.png
[debug] pid:25204   : Downloaded icon file: E:\github\hwa-starter\tmp\StarterPack\cordova\img\icons\Icon@170w.png
[debug] pid:25204   : Downloaded icon file: E:\github\hwa-starter\tmp\StarterPack\cordova\img\icons\Icon@270w.png
[debug] pid:25204   : Downloaded icon file: E:\github\hwa-starter\tmp\StarterPack\cordova\img\icons\Icon@256w.png
[debug] pid:25204   : Downloaded icon file: E:\github\hwa-starter\tmp\StarterPack\cordova\img\icons\Splash.png
[debug] pid:25204   : Downloaded icon file: E:\github\hwa-starter\tmp\StarterPack\cordova\img\icons\Icon@360w.png
[debug] pid:25204   : Removing default images from Cordova configuration...
[debug] pid:25204   : Copying wrapper html file for the windows platform from E:\github\hwa-starter\tmp\StarterPack\cordova\plugins\cordova-plugin-hostedwebapp\assets\windows\wrapper.html to E:\github\hwa-starter\tmp\StarterPack\cordova\platforms\windows\www\wrapper.html.
[debug] pid:25204   : Finished copying wrapper html file for the windows platform.
[debug] pid:25204   : Copying wrapper js file for the windows platform from E:\github\hwa-starter\tmp\StarterPack\cordova\plugins\cordova-plugin-hostedwebapp\assets\windows\wrapper.js to E:\github\hwa-starter\tmp\StarterPack\cordova\platforms\windows\www\js\wrapper.js.
[debug] pid:25204   : Finished copying wrapper js file for the windows platform.
[debug] pid:25204   : Copying wrapper css file for the windows platform from E:\github\hwa-starter\tmp\StarterPack\cordova\plugins\cordova-plugin-hostedwebapp\assets\windows\wrapper.css to E:\github\hwa-starter\tmp\StarterPack\cordova\platforms\windows\www\css\wrapper.css.
[debug] pid:25204   : Finished copying wrapper css file for the windows platform.
[debug] pid:25204   : Installing "cordova-plugin-hostedwebapp" for windows
[debug] pid:25204   : Installing "cordova-plugin-whitelist" for windows
[debug] pid:25204   : Fetching plugin "cordova-plugin-network-information" via npm
[debug] pid:25204   : Installing "cordova-plugin-network-information" for windows
[debug] pid:25204   : Plugin "cordova-plugin-whitelist" already installed on windows. Making it top-level.
[info ] cordova     : Processing the 'windows' Cordova platform...
[info ] cordova     : Copying documentation from 'C:\Users\zackhall\AppData\Roaming\npm\node_modules\manifoldjs\node_modules\manifoldjs-cordova\lib\docs\windows' to 'E:\github\hwa-starter\tmp\StarterPack\cordova\platforms\windows'...
[info ] cordova     : Writing the generation information for the 'Cordova Platform' platform to 'E:\github\hwa-starter\tmp\StarterPack\cordova\platforms\windows\generationInfo.json'...
[info ] cordova     : Writing the generation information for the 'Cordova Platform' platform to 'E:\github\hwa-starter\tmp\StarterPack\cordova\generationInfo.json'...
[info ] manifoldjs  : The Cordova Platform app was created successfully!
**[warn ] manifoldjs  : One or more platforms could not be built successfully. Correct any errors and then run manifoldjs package [project-directory] [options] to build the applications.**
[info ] manifoldjs  : The application(s) are ready.

The only indication of an issue I see is the line I bolded above. There's no debuggable output even with debug enabled.

When I check the output I see that the Windows folder isn't there and I can't navigate and run "manifoldjs run windows".

zackhall commented 8 years ago

Spoke with @boyofgreen. The command should be using windows10 as the platform name, and not windows. Documentation on ManifoldJS website is going to be updated to reflect this. Using windows10 I was able to build and run no problem.