mirror of
				https://github.com/WarrenHood/MCModpackManager.git
				synced 2025-11-04 01:58:41 +00:00 
			
		
		
		
	Improve logging when we can't find a package
This commit is contained in:
		
							parent
							
								
									ec50a380b8
								
							
						
					
					
						commit
						844350ab09
					
				| 
						 | 
					@ -6,7 +6,7 @@ mod resolver;
 | 
				
			||||||
use clap::{Parser, Subcommand};
 | 
					use clap::{Parser, Subcommand};
 | 
				
			||||||
use mod_meta::{ModMeta, ModProvider};
 | 
					use mod_meta::{ModMeta, ModProvider};
 | 
				
			||||||
use modpack::ModpackMeta;
 | 
					use modpack::ModpackMeta;
 | 
				
			||||||
use std::{borrow::BorrowMut, error::Error, path::PathBuf};
 | 
					use std::{error::Error, path::PathBuf};
 | 
				
			||||||
 | 
					
 | 
				
			||||||
/// A Minecraft Modpack Manager
 | 
					/// A Minecraft Modpack Manager
 | 
				
			||||||
#[derive(Parser)]
 | 
					#[derive(Parser)]
 | 
				
			||||||
| 
						 | 
					
 | 
				
			||||||
| 
						 | 
					@ -64,9 +64,12 @@ impl Modrinth {
 | 
				
			||||||
        versions.reverse();
 | 
					        versions.reverse();
 | 
				
			||||||
 | 
					
 | 
				
			||||||
        let package = if mod_meta.version == "*" {
 | 
					        let package = if mod_meta.version == "*" {
 | 
				
			||||||
            versions
 | 
					            versions.last().ok_or(format!(
 | 
				
			||||||
                .last()
 | 
					                "Cannot find package {} for loader={} and mc version={}",
 | 
				
			||||||
                .ok_or(format!("Cannot find package {}", mod_meta.name))?
 | 
					                mod_meta.name,
 | 
				
			||||||
 | 
					                pack_meta.modloader.to_string().to_lowercase(),
 | 
				
			||||||
 | 
					                pack_meta.mc_version
 | 
				
			||||||
 | 
					            ))?
 | 
				
			||||||
        } else {
 | 
					        } else {
 | 
				
			||||||
            let expected_version = semver::Version::parse(&mod_meta.version)?;
 | 
					            let expected_version = semver::Version::parse(&mod_meta.version)?;
 | 
				
			||||||
            versions
 | 
					            versions
 | 
				
			||||||
| 
						 | 
					
 | 
				
			||||||
| 
						 | 
					@ -99,6 +99,11 @@ impl PinnedPackMeta {
 | 
				
			||||||
                                .collect());
 | 
					                                .collect());
 | 
				
			||||||
                        }
 | 
					                        }
 | 
				
			||||||
                        return Ok(vec![]);
 | 
					                        return Ok(vec![]);
 | 
				
			||||||
 | 
					                    } else if let Err(e) = pinned_mod {
 | 
				
			||||||
 | 
					                        eprintln!(
 | 
				
			||||||
 | 
					                            "Failed to resolve {}@{} with provider {:#?}: {}",
 | 
				
			||||||
 | 
					                            mod_metadata.name, mod_metadata.version, mod_provider, e
 | 
				
			||||||
 | 
					                        );
 | 
				
			||||||
                    }
 | 
					                    }
 | 
				
			||||||
                }
 | 
					                }
 | 
				
			||||||
                crate::mod_meta::ModProvider::Raw => unimplemented!(),
 | 
					                crate::mod_meta::ModProvider::Raw => unimplemented!(),
 | 
				
			||||||
| 
						 | 
					
 | 
				
			||||||
		Loading…
	
		Reference in a new issue