feat: use full URL for embedded script paths

Signed-off-by: Rustin170506 <[email protected]>

Author: 0xPoe

src/cargo/core/workspace.rs

@@ -1800,7 +1800,10 @@ impl<'gctx> Packages<'gctx> {
         match self.packages.entry(key.to_path_buf()) {
             Entry::Occupied(e) => Ok(e.into_mut()),
             Entry::Vacant(v) => {
-                let source_id = SourceId::for_path(key)?;
+                let source_id = match crate::util::toml::is_embedded(manifest_path) {
+                    true => SourceId::for_path(manifest_path)?,
+                    false => SourceId::for_path(key)?,
+                };
                 let manifest = read_manifest(manifest_path, source_id, self.gctx)?;
                 Ok(v.insert(match manifest {
                     EitherManifest::Real(manifest) => {

tests/testsuite/script.rs

@@ -1307,11 +1307,10 @@ fn cmd_pkgid_with_embedded() {
         .masquerade_as_nightly_cargo(&["script"])
         .run();
 
-    // FIXME: It should be `path+[ROOTURL]/foo/script.rs#[email protected]`.
     p.cargo("-Zscript pkgid --manifest-path script.rs")
         .masquerade_as_nightly_cargo(&["script"])
         .with_stdout_data(str![[r#"
-path+[ROOTURL]/foo#[email protected]
+path+[ROOTURL]/foo/script.rs#[email protected]
 
 "#]])
         .with_stderr_data(str![[r#"