cerc-io/lighthouse
16
0
Fork 0
Code Issues 1 Pull Requests Packages Projects Releases Wiki Activity

Return 32-bit architecture error earlier (#1356)

Browse Source 
## Issue Addressed

NA

## Proposed Changes

Exits *earlier* when system is 32-bit. Previously we were trying (and failing) to parse the genesis SSZ first. This made for a bad user experience.

## Additional Info

NA
This commit is contained in:
Paul Hauner 2020-07-14 06:23:33 +00:00
parent 3c7f2d651a
commit 00c89c51c8
1 changed files with 7 additions and 9 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

16
lighthouse/src/main.rs
View File

@ -156,6 +156,13 @@ fn run<E: EthSpec>(
environment_builder: EnvironmentBuilder<E>, environment_builder: EnvironmentBuilder<E>,
matches: &ArgMatches, matches: &ArgMatches,
) -> Result<(), String> { ) -> Result<(), String> {
if std::mem::size_of::<usize>() != 8 {
return Err(format!(
"{}bit architecture is not supported (64bit only).",
std::mem::size_of::<usize>() * 8
));
}
let debug_level = matches let debug_level = matches
.value_of("debug-level") .value_of("debug-level")
.ok_or_else(|| "Expected --debug-level flag".to_string())?; .ok_or_else(|| "Expected --debug-level flag".to_string())?;
@ -180,15 +187,6 @@ fn run<E: EthSpec>(
environment.log_to_json_file(path, debug_level, log_format)?; environment.log_to_json_file(path, debug_level, log_format)?;
} }
if std::mem::size_of::<usize>() != 8 {
crit!(
log,
"Lighthouse only supports 64bit CPUs";
"detected" => format!("{}bit", std::mem::size_of::<usize>() * 8)
);
return Err("Invalid CPU architecture".into());
}
// Note: the current code technically allows for starting a beacon node _and_ a validator // Note: the current code technically allows for starting a beacon node _and_ a validator
// client at the same time. // client at the same time.
// //