@@ -6,8 +6,11 @@ use std::{
66use url:: Url ;
77
88use super :: helpers:: download_file:: download_file;
9- use crate :: run:: { check_system:: SystemInfo , config:: Config } ;
109use crate :: { prelude:: * , MONGODB_TRACER_VERSION , VALGRIND_CODSPEED_VERSION } ;
10+ use crate :: {
11+ run:: { check_system:: SystemInfo , config:: Config } ,
12+ VALGRIND_CODSPEED_DEB_VERSION ,
13+ } ;
1114
1215/// Run a command with sudo if available
1316fn run_with_sudo ( command_args : & [ & str ] ) -> Result < ( ) > {
@@ -44,18 +47,18 @@ fn get_codspeed_valgrind_filename(system_info: &SystemInfo) -> Result<String> {
4447 system_info. os_version . as_str ( ) ,
4548 system_info. arch . as_str ( ) ,
4649 ) {
47- ( "ubuntu" , "20.04" , "x86_64" ) | ( "debian" , "11" , "x86_64" ) | ( "debian" , "12" , "x86_64" ) => {
48- ( "20.04" , "amd64" )
49- }
50- ( "ubuntu" , "22.04" , "x86_64" ) => ( "22.04" , "amd64" ) ,
50+ ( "ubuntu" , "22.04" , "x86_64" ) | ( "debian" , "12" , "x86_64" ) => ( "22.04" , "amd64" ) ,
5151 ( "ubuntu" , "24.04" , "x86_64" ) => ( "24.04" , "amd64" ) ,
52- ( "ubuntu" , "22.04" , "aarch64" ) => ( "22.04" , "arm64" ) ,
52+ ( "ubuntu" , "22.04" , "aarch64" ) | ( "debian" , "12" , "aarch64" ) => ( "22.04" , "arm64" ) ,
53+ ( "ubuntu" , "24.04" , "aarch64" ) => ( "24.04" , "arm64" ) ,
5354 _ => bail ! ( "Unsupported system" ) ,
5455 } ;
5556
5657 Ok ( format ! (
5758 "valgrind_{}_ubuntu-{}_{}.deb" ,
58- VALGRIND_CODSPEED_VERSION , version, architecture
59+ VALGRIND_CODSPEED_DEB_VERSION . as_str( ) ,
60+ version,
61+ architecture
5962 ) )
6063}
6164
@@ -74,8 +77,7 @@ fn is_valgrind_installed() -> bool {
7477 }
7578
7679 let version = String :: from_utf8_lossy ( & version_output. stdout ) ;
77- // TODO: use only VALGRIND_CODSPEED_VERSION here, the other value is when valgrind has been built locally
78- version. contains ( "valgrind-3.21.0.codspeed" ) || version. contains ( VALGRIND_CODSPEED_VERSION )
80+ version. contains ( VALGRIND_CODSPEED_VERSION )
7981 } else {
8082 false
8183 }
@@ -155,7 +157,7 @@ mod tests {
155157 } ;
156158 assert_snapshot ! (
157159 get_codspeed_valgrind_filename( & system_info) . unwrap( ) ,
158- @"valgrind_3.21 .0-0codspeed3_ubuntu -22.04_amd64.deb"
160+ @"valgrind_3.24 .0-0codspeed1_ubuntu -22.04_amd64.deb"
159161 ) ;
160162 }
161163
@@ -169,21 +171,21 @@ mod tests {
169171 } ;
170172 assert_snapshot ! (
171173 get_codspeed_valgrind_filename( & system_info) . unwrap( ) ,
172- @"valgrind_3.21 .0-0codspeed3_ubuntu -24.04_amd64.deb"
174+ @"valgrind_3.24 .0-0codspeed1_ubuntu -24.04_amd64.deb"
173175 ) ;
174176 }
175177
176178 #[ test]
177179 fn test_system_info_to_codspeed_valgrind_version_debian ( ) {
178180 let system_info = SystemInfo {
179181 os : "debian" . to_string ( ) ,
180- os_version : "11 " . to_string ( ) ,
182+ os_version : "12 " . to_string ( ) ,
181183 arch : "x86_64" . to_string ( ) ,
182184 ..SystemInfo :: test ( )
183185 } ;
184186 assert_snapshot ! (
185187 get_codspeed_valgrind_filename( & system_info) . unwrap( ) ,
186- @"valgrind_3.21 .0-0codspeed3_ubuntu-20 .04_amd64.deb"
188+ @"valgrind_3.24 .0-0codspeed1_ubuntu-22 .04_amd64.deb"
187189 ) ;
188190 }
189191
@@ -197,7 +199,7 @@ mod tests {
197199 } ;
198200 assert_snapshot ! (
199201 get_codspeed_valgrind_filename( & system_info) . unwrap( ) ,
200- @"valgrind_3.21 .0-0codspeed3_ubuntu -22.04_arm64.deb"
202+ @"valgrind_3.24 .0-0codspeed1_ubuntu -22.04_arm64.deb"
201203 ) ;
202204 }
203205}
0 commit comments