Script 'mail_helper' called by obssrc Hello community, here is the log from the commit of package wayshot for openSUSE:Factory checked in at 2025-08-17 14:50:05 ++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++ Comparing /work/SRC/openSUSE:Factory/wayshot (Old) and /work/SRC/openSUSE:Factory/.wayshot.new.1085 (New) ++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++
Package is "wayshot" Sun Aug 17 14:50:05 2025 rev:17 rq:1299950 version:1.3.1+git62.g4f130921361abcbf23159478ebb16c25c808f442 Changes: -------- --- /work/SRC/openSUSE:Factory/wayshot/wayshot.changes 2025-07-17 17:20:55.761158237 +0200 +++ /work/SRC/openSUSE:Factory/.wayshot.new.1085/wayshot.changes 2025-08-17 14:50:24.220662153 +0200 @@ -1,0 +2,11 @@ +Sat Aug 16 23:26:33 UTC 2025 - Soc Virnyl Estela <uncomfyhaloma...@opensuse.org> + +- Update to version 1.3.1+git62.g4f130921361abcbf23159478ebb16c25c808f442: + * clippy: apply "collapsible if" suggestion here too + * clippy: apply more "collapsible if" suggestions + * clippy: apply "collapsible if" suggestion + * clippy: apply "collapsible if" suggestion + * clippy: apply clippy fixes + * fix: logic for argument `-` will not produce a file at the current directory + +------------------------------------------------------------------- Old: ---- wayshot-1.3.1+git56.g28331dcc3886f8b9e6bc09f9951fe59744c16acf.tar.zst New: ---- wayshot-1.3.1+git62.g4f130921361abcbf23159478ebb16c25c808f442.tar.zst ++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++ Other differences: ------------------ ++++++ wayshot.spec ++++++ --- /var/tmp/diff_new_pack.dUOuh6/_old 2025-08-17 14:50:24.820687028 +0200 +++ /var/tmp/diff_new_pack.dUOuh6/_new 2025-08-17 14:50:24.820687028 +0200 @@ -1,7 +1,7 @@ # # spec file for package wayshot # -# Copyright (c) 2025 SUSE LLC +# Copyright (c) 2025 SUSE LLC and contributors # # All modifications and additions to the file contributed by third parties # remain the property of their copyright owners, unless otherwise agreed @@ -19,7 +19,7 @@ %global rustflags '-Clink-arg=-Wl,-z,relro,-z,now' Name: wayshot -Version: 1.3.1+git56.g28331dcc3886f8b9e6bc09f9951fe59744c16acf +Version: 1.3.1+git62.g4f130921361abcbf23159478ebb16c25c808f442 Release: 0 Summary: Screenshot tool for wlroots based compositors License: (0BSD OR MIT OR Apache-2.0) AND (Apache-2.0 OR MIT) AND (Apache-2.0 OR MIT) AND (Apache-2.0 OR MIT OR Zlib) AND (MIT OR Unlicense) AND (Apache-2.0 OR Zlib OR MIT) AND BSD-3-Clause AND ISC AND MIT AND Zlib AND BSD-2-Clause ++++++ vendor.tar.zst ++++++ /work/SRC/openSUSE:Factory/wayshot/vendor.tar.zst /work/SRC/openSUSE:Factory/.wayshot.new.1085/vendor.tar.zst differ: char 7, line 1 ++++++ wayshot-1.3.1+git56.g28331dcc3886f8b9e6bc09f9951fe59744c16acf.tar.zst -> wayshot-1.3.1+git62.g4f130921361abcbf23159478ebb16c25c808f442.tar.zst ++++++ diff -urN '--exclude=CVS' '--exclude=.cvsignore' '--exclude=.svn' '--exclude=.svnignore' old/wayshot-1.3.1+git56.g28331dcc3886f8b9e6bc09f9951fe59744c16acf/libwayshot/examples/waymirror-egl/src/dispatch.rs new/wayshot-1.3.1+git62.g4f130921361abcbf23159478ebb16c25c808f442/libwayshot/examples/waymirror-egl/src/dispatch.rs --- old/wayshot-1.3.1+git56.g28331dcc3886f8b9e6bc09f9951fe59744c16acf/libwayshot/examples/waymirror-egl/src/dispatch.rs 1970-01-01 01:00:00.000000000 +0100 +++ new/wayshot-1.3.1+git62.g4f130921361abcbf23159478ebb16c25c808f442/libwayshot/examples/waymirror-egl/src/dispatch.rs 1970-01-01 01:00:00.000000000 +0100 @@ -143,10 +143,9 @@ if let wl_seat::Event::Capabilities { capabilities: WEnum::Value(capabilities), } = event + && capabilities.contains(wl_seat::Capability::Keyboard) { - if capabilities.contains(wl_seat::Capability::Keyboard) { - seat.get_keyboard(qh, ()); - } + seat.get_keyboard(qh, ()); } } } @@ -160,11 +159,11 @@ _: &Connection, _: &QueueHandle<Self>, ) { - if let wl_keyboard::Event::Key { key, .. } = event { - if key == 1 { - // ESC key - state.running = false; - } + if let wl_keyboard::Event::Key { key, .. } = event + && key == 1 + { + // ESC key + state.running = false; } } } diff -urN '--exclude=CVS' '--exclude=.cvsignore' '--exclude=.svn' '--exclude=.svnignore' old/wayshot-1.3.1+git56.g28331dcc3886f8b9e6bc09f9951fe59744c16acf/libwayshot/src/dispatch.rs new/wayshot-1.3.1+git62.g4f130921361abcbf23159478ebb16c25c808f442/libwayshot/src/dispatch.rs --- old/wayshot-1.3.1+git56.g28331dcc3886f8b9e6bc09f9951fe59744c16acf/libwayshot/src/dispatch.rs 1970-01-01 01:00:00.000000000 +0100 +++ new/wayshot-1.3.1+git62.g4f130921361abcbf23159478ebb16c25c808f442/libwayshot/src/dispatch.rs 1970-01-01 01:00:00.000000000 +0100 @@ -72,21 +72,20 @@ interface, version, } = event + && interface == "wl_output" { - if interface == "wl_output" { - if version >= 4 { - let output = wl_registry.bind::<wl_output::WlOutput, _, _>(name, 4, qh, ()); - state.outputs.push(OutputInfo { - wl_output: output, - name: "".to_string(), - description: String::new(), - transform: wl_output::Transform::Normal, - physical_size: Size::default(), - logical_region: LogicalRegion::default(), - }); - } else { - tracing::error!("Ignoring a wl_output with version < 4."); - } + if version >= 4 { + let output = wl_registry.bind::<wl_output::WlOutput, _, _>(name, 4, qh, ()); + state.outputs.push(OutputInfo { + wl_output: output, + name: "".to_string(), + description: String::new(), + transform: wl_output::Transform::Normal, + physical_size: Size::default(), + logical_region: LogicalRegion::default(), + }); + } else { + tracing::error!("Ignoring a wl_output with version < 4."); } } } diff -urN '--exclude=CVS' '--exclude=.cvsignore' '--exclude=.svn' '--exclude=.svnignore' old/wayshot-1.3.1+git56.g28331dcc3886f8b9e6bc09f9951fe59744c16acf/libwayshot/src/lib.rs new/wayshot-1.3.1+git62.g4f130921361abcbf23159478ebb16c25c808f442/libwayshot/src/lib.rs --- old/wayshot-1.3.1+git56.g28331dcc3886f8b9e6bc09f9951fe59744c16acf/libwayshot/src/lib.rs 1970-01-01 01:00:00.000000000 +0100 +++ new/wayshot-1.3.1+git62.g4f130921361abcbf23159478ebb16c25c808f442/libwayshot/src/lib.rs 1970-01-01 01:00:00.000000000 +0100 @@ -165,7 +165,7 @@ tracing::error!( "Failed to create ZxdgOutputManagerV1 version 3. Does your compositor implement ZxdgOutputManagerV1?" ); - panic!("{:#?}", e); + panic!("{e:#?}"); } }; @@ -745,7 +745,7 @@ } /// Get a FrameCopy instance with screenshot pixel data for any wl_output object. - #[tracing::instrument(skip_all, fields(output = format!("{output_info}"), region = capture_region.map(|r| format!("{:}", r)).unwrap_or("fullscreen".to_string())))] + #[tracing::instrument(skip_all, fields(output = format!("{output_info}"), region = capture_region.map(|r| format!("{r:}")).unwrap_or("fullscreen".to_string())))] fn capture_frame_copy( &self, cursor_overlay: bool, diff -urN '--exclude=CVS' '--exclude=.cvsignore' '--exclude=.svn' '--exclude=.svnignore' old/wayshot-1.3.1+git56.g28331dcc3886f8b9e6bc09f9951fe59744c16acf/wayshot/build.rs new/wayshot-1.3.1+git62.g4f130921361abcbf23159478ebb16c25c808f442/wayshot/build.rs --- old/wayshot-1.3.1+git56.g28331dcc3886f8b9e6bc09f9951fe59744c16acf/wayshot/build.rs 1970-01-01 01:00:00.000000000 +0100 +++ new/wayshot-1.3.1+git62.g4f130921361abcbf23159478ebb16c25c808f442/wayshot/build.rs 1970-01-01 01:00:00.000000000 +0100 @@ -14,10 +14,9 @@ .stdout(Stdio::null()) .stderr(Stdio::null()) .spawn() + && let ErrorKind::NotFound = e.kind() { - if let ErrorKind::NotFound = e.kind() { - return Ok(()); - } + return Ok(()); } // We just append "out" so it's easy to find all the scdoc output later in line 38. diff -urN '--exclude=CVS' '--exclude=.cvsignore' '--exclude=.svn' '--exclude=.svnignore' old/wayshot-1.3.1+git56.g28331dcc3886f8b9e6bc09f9951fe59744c16acf/wayshot/src/utils.rs new/wayshot-1.3.1+git62.g4f130921361abcbf23159478ebb16c25c808f442/wayshot/src/utils.rs --- old/wayshot-1.3.1+git56.g28331dcc3886f8b9e6bc09f9951fe59744c16acf/wayshot/src/utils.rs 1970-01-01 01:00:00.000000000 +0100 +++ new/wayshot-1.3.1+git62.g4f130921361abcbf23159478ebb16c25c808f442/wayshot/src/utils.rs 1970-01-01 01:00:00.000000000 +0100 @@ -148,7 +148,7 @@ pub fn get_default_file_name(filename_format: &str, encoding: EncodingFormat) -> PathBuf { let format = Local::now().format(filename_format); - PathBuf::from(format!("{}.{}", format, encoding)) + PathBuf::from(format!("{format}.{encoding}")) } pub fn get_full_file_name(path: &Path, filename_format: &str, encoding: EncodingFormat) -> PathBuf { @@ -166,6 +166,6 @@ .file_stem() .unwrap_or_default() .to_string_lossy(); - base_dir.join(format!("{}.{}", stem, encoding)) + base_dir.join(format!("{stem}.{encoding}")) } } diff -urN '--exclude=CVS' '--exclude=.cvsignore' '--exclude=.svn' '--exclude=.svnignore' old/wayshot-1.3.1+git56.g28331dcc3886f8b9e6bc09f9951fe59744c16acf/wayshot/src/wayshot.rs new/wayshot-1.3.1+git62.g4f130921361abcbf23159478ebb16c25c808f442/wayshot/src/wayshot.rs --- old/wayshot-1.3.1+git56.g28331dcc3886f8b9e6bc09f9951fe59744c16acf/wayshot/src/wayshot.rs 1970-01-01 01:00:00.000000000 +0100 +++ new/wayshot-1.3.1+git62.g4f130921361abcbf23159478ebb16c25c808f442/wayshot/src/wayshot.rs 1970-01-01 01:00:00.000000000 +0100 @@ -65,12 +65,12 @@ .or(input_encoding) .unwrap_or(file.encoding.unwrap_or_default()); - if let Some(ie) = input_encoding { - if ie != encoding { - tracing::warn!( - "The encoding requested '{encoding}' does not match the output file's encoding '{ie}'. Still using the requested encoding however.", - ); - } + if let Some(ie) = input_encoding + && ie != encoding + { + tracing::warn!( + "The encoding requested '{encoding}' does not match the output file's encoding '{ie}'. Still using the requested encoding however.", + ); } let file_name_format = cli.file_name_format.unwrap_or( @@ -163,14 +163,14 @@ let mut image_buf: Option<Cursor<Vec<u8>>> = None; if let Some(f) = file { - image_buffer.save(f)?; - } - - if stdout_print { - let mut buffer = Cursor::new(Vec::new()); - image_buffer.write_to(&mut buffer, encoding.into())?; - writer.write_all(buffer.get_ref())?; - image_buf = Some(buffer); + if stdout_print { + let mut buffer = Cursor::new(Vec::new()); + image_buffer.write_to(&mut buffer, encoding.into())?; + writer.write_all(buffer.get_ref())?; + image_buf = Some(buffer); + } else { + image_buffer.save(f)?; + } } if clipboard {