Titanium JIRA Archive
Titanium SDK/CLI (TIMOB)

[TIMOB-19134] CLI: The reported architecture in the information banner is for the node process rather than the OS architecture

GitHub Issuen/a
TypeBug
PriorityNone
StatusOpen
ResolutionUnresolved
Affected Version/sRelease 4.0.0, Release 4.1.0
Fix Version/sn/a
ComponentsCLI
Labelscb-tooling, qe-4.1.0
ReporterEwan Harris
AssigneePraveen Innamuri
Created2015-07-01T17:24:59.000+0000
Updated2020-01-30T10:48:03.000+0000

Description

Description

The architecture reported in the information banner on build is for the architecture of Node rather than the OS architecture. For example if I run 32-bit Node on 64-bit Windows then the following is reported
 Operating System 
Name = Microsoft Windows 8.1 Enterprise N 
Version = 6.3.9600 
Architecture = 32bit 

CPUs = 2

Memory = 3756511232

[This is a Node issue](https://github.com/joyent/node/issues/2862)

Steps To Reproduce

1. On a 64-bit Windows machine install 32-bit Node 2. Run an build comand to show the banner appc run -p android -T device --build-only

Actual Result

32-bit is shown as the architecture

Expected Result

64-bit should be shown

Comments

  1. Ewan Harris 2018-07-10

    [~cbarber] Do you think it's worth bringing the [arch function from appcd-utils](https://github.com/appcelerator/appc-daemon/blob/42198d967bf2792d93bfcee8f1ef4bcacaad8969/packages/appcd-util/src/util.js#L29-L49) in to this (even it's a ES5 rewrite into node-appc/node-titanium-sdk), or should we close as wont fix/not our bug?
  2. Chris Barber 2018-07-10

    I'm not sure what prints the "banner" in the description. Being that the system info in the daemon is correct, I'm not sure it's worth addressing in the current CLI's. Furthermore, getting the system architecture is not cheap. We have to spawn a process on Linux. So, I vote to not fix this in the current tooling and we'll expose the correct value from the daemon in the next Titanium CLI.
  3. Ewan Harris 2018-07-11

    It comes from, [here in the build](https://github.com/appcelerator/titanium_mobile/blob/1321070126d9fa09cdb1bab2ef92573fe7aab0b7/cli/commands/build.js#L442), which uses [this function in node-appc](https://github.com/appcelerator/node-appc/blob/5e92ca0768343d74ef39d439a568ac38db58a9e8/lib/environ.js#L223), I do see value in fixing this in current CLIs as it would potentially improve analytics data we have on what architectures people are using, maybe we could ignore linux (I doubt maybe people are running 32-bit on 64-bit systems there), I just want to try fix the situation where we're reporting all Studio users as 32-bit as that's what Studio installs
  4. Chris Barber 2018-07-11

    Then go for it.

JSON Source