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

Should CLI only run in 64 bit mode? #39

Closed
felix-b opened this issue Apr 6, 2017 · 2 comments
Closed

Should CLI only run in 64 bit mode? #39

felix-b opened this issue Apr 6, 2017 · 2 comments

Comments

@felix-b
Copy link
Member

felix-b commented Apr 6, 2017

Symptom: when debugging in Visual Studio on 64 bit Windows, the process runs in 32 bit mode.

Cause: NWheels.Cli project defines Platform=Any CPU. Since Visual Studio is a 32 bit process, Any CPU defaults to 32 bit mode.

Fix: if we want to always run in 64 bit, we need to set Platform=x64 for NWheels.Cli.

@felix-b felix-b added this to the Backlog milestone Apr 6, 2017
@felix-b
Copy link
Member Author

felix-b commented Apr 6, 2017

The question is, can we say that we only run 64 bit microservices?

There may be a use case when a microservice has to load a 32-bit 3rd party component. If we do what I suggested (Platform=x64), we block this use case.

@felix-b felix-b mentioned this issue Apr 6, 2017
Merged
felix-b added a commit that referenced this issue Apr 6, 2017
@felix-b
Copy link
Member Author

felix-b commented Apr 6, 2017

The following is an offline discussion between me and @ILyaIL83:

@ILyaIL83: I'm not sure. On one side, who needs 32 bit processes today.
@me: (I mention two projects where we had to load 32 bit components in-process)
@ILyaIL83: On the other side, we can continue developing on 32 bit, and it will run on 64 bit, then we can leave everything as is.
@me: I think we have to be able to run in both 32 and 64. But there needs to be a way to choose.
@ILyaIL83: Yes, I thought about it, too
@me: the question is, how we can make F5 run in 64 bit by default
@ILyaIL83: I'm also thinking about creating integration test for both 32 and 64 bit, but maybe it's an overkill
@me: I think we should have such tests
@ILyaIL83: Well, by now I have more questions than answers. Which means we should put it on hold, until the problem matures.
@me: I agree.

@felix-b felix-b added epic and removed bug labels Apr 6, 2017
@felix-b felix-b changed the title CLI should run in 64 bit mode Should CLI only run in 64 bit mode? Apr 6, 2017
@felix-b felix-b modified the milestones: Proposals, Backlog Apr 20, 2017
@felix-b felix-b closed this as completed Mar 29, 2019
@felix-b felix-b added the gen1 label Mar 29, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

1 participant