ci: When autopublishing, remove xtask from workspace

[ShoyuVanilla]

c.c #19903

[lnicola]

Should we also revert the other change, then?

[ShoyuVanilla]

Should we also revert the other change, then?

Yes, I think that's reasonable

[lnicola]

I need to do it this way because there's no space before xtask. The order doesn't matter though.

Suggested change

	sed -i 's/ "lib\/\*",//; s/ "xtask\/",//' ./Cargo.toml
	sed -i 's/"xtask\\/",//; s/ "lib\/\*",//' ./Cargo.toml

[ShoyuVanilla]

Oh, good catch!

[ShoyuVanilla]

sed: -e expression #1, char 15: unknown option to `s'

Argh, sed in CI is not receiving ; 😢
The escape character was wrong. Sorry

[lnicola]

sed -i 's|"xtask/", "lib/\*", ||' Cargo.toml works, but we can also take this from the other workflow:

          # Only publish the crates under lib/
          sed -i 's|^members = .*$|members = ["lib/*"]|' Cargo.toml

I was sure I made this change before, that's why.

[lnicola]

I see now that sed -i 's/"xtask\\/",//; s/ "lib\/\*",//' ./Cargo.toml is missing a \, but it actually worked for me in fish.

EDIT: no, not missing, an extra one.

[ShoyuVanilla]

Yes, that seems better.

sed -i 's/"xtask\\/",//; s/ "lib\/\*",//' ./Cargo.toml
                ^ This redundant backslash is the problem

[ShoyuVanilla]

Opened #19920 😅