Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Improve Windows deploy VBScript. #8

Closed
wants to merge 1 commit into from
Closed

Improve Windows deploy VBScript. #8

wants to merge 1 commit into from

Conversation

tabad
Copy link
Contributor

@tabad tabad commented Aug 15, 2013

This morning I published a post into forum.fusioninventory.org that resolves the problem of deploy the new FusionInventory Agent 2.3.0 with the new FusionInventory Agent installer for Microsoft Windows while maintaining compatibility with the previous FusionInventory Agent installer.

This is the patch file. I have tested it but would be well do more testing before applying it.

@mipsou
Copy link
Contributor

mipsou commented Aug 16, 2013

Must be added the detection architecture for the auto mode
It is the registry or system variable as below.
http://i.snag.gy/y4HQf.jpg
For my part, I park next to a heterogeneous architecture. So I'm forced to have the auto mode.
It will take work to add autodetect mode.

@+

[code]
string arch = System.Environment.GetEnvironmentVariable("PROCESSOR_ARCHITECTURE",
EnvironmentVariableTarget.Machine);
[/code]

@goneri
Copy link
Contributor

goneri commented Aug 16, 2013

Hi @mips81 and @tabad ,

I merged the patch. @mips81, I don't understand what your want to change. Can you provide a patch?

Best regards

@mipsou
Copy link
Contributor

mipsou commented Aug 16, 2013

i would give the choice to the user betwin Autodetect and forced

In line 87

I would add "autodetect" to the Setup_Architecture variable and getting
Processor architecture

For exemple:

.
.
.
.
SetupArchitecture = "Auto"
.
.
.
Select Case SetupArchitecture
Case SetupArchitecture "Auto"
Call DetectArch()
Call Install(Processor_Architecture)
Case SetupArchitecture "AMD64"
Call Install(x64)
Case SetupArchitecture "x86"
Case Install(x86)
Case Else
Call Error("Unknow")

King regards

2013/8/16 Gonéri Le Bouder notifications@github.com

Hi @mips81 https://github.com/mips81 and @tabadhttps://github.com/tabad,

I merged the patch. @mips81 https://github.com/mips81, I don't
understand what your want to change. Can you provide a patch?

Best regards


Reply to this email directly or view it on GitHubhttps://github.com//pull/8#issuecomment-22772599
.

Christophe PUJOL

CV: http://www.doyoubuzz.com/christophe-pujol_0

@goneri goneri closed this Aug 16, 2013
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants