[Merge] ~dviererbe/ubuntu-seeds/+git/ubuntu:add-dotnet8-to-noble into ~ubuntu-core-dev/ubuntu-seeds/+git/ubuntu:noble

Lukas Märdian mp+463930 at code.launchpad.net
Wed Apr 10 10:32:12 UTC 2024


Review: Needs Fixing

Plenty of inline comments! :)

I guess we can reduce it to just this:

 * dotnet8  # MIR (LP: #2060056)
 * aspnetcore-runtime-dbg-8.0
 * dotnet-runtime-dbg-8.0
 * dotnet-sdk-dbg-8.0


I wonder where the *-dbg-8.0 naming comes from? Would it be *-dbg, it could be caught by "Extra-Includes" automatically. But I guess it's fine to list those 3 packages here explicitly. Are the *-dbg-* packages really needed in main, though?

The original dotNET6 MIR review didn't mention them explicitly: "List of specific binary packages to be promoted to main: dotnet6 dotnet-host dotnet-hostfxr-6.0 dotnet-runtime-6.0 aspnetcore-runtime-6.0 dotnet-templates-6.0 dotnet-sdk-6.0 dotnet-targeting-pack-6.0 netstandard-targeting-pack-2.1 aspnetcore-targeting-pack-6.0 dotnet-apphost-pack-6.0"

IMO it should be fine either way. Keep or drop *-dbg-* as you like!

Diff comments:

> diff --git a/development b/development
> index c448b52..eadb445 100644
> --- a/development
> +++ b/development
> @@ -43,3 +43,19 @@ Additional supported languages
>   * golang
>   * rustc
>   * cargo
> +

After reducing all the unneeded lines, we can probably drop this empty line, too. It will be a much smaller dotNET8 block :)

> + * aspnetcore-runtime-8.0

dotnet8 -> dotnet-sdk-8.0 "Depends" on this, should be dropped from the seed.

> + * aspnetcore-runtime-dbg-8.0

Note: This is not covered by Extra-Include, so it's fine to stay here.

> + * aspnetcore-targeting-pack-8.0

dotnet8 -> dotnet-sdk-8.0 "Depends" on this, should be dropped from the seed.

> + * dotnet-apphost-pack-8.0

dotnet8 -> dotnet-sdk-8.0 "Depends" on this, should be dropped from the seed.

> + * dotnet-host-8.0

dotnet8 -> dotnet-sdk-8.0 -> dotnet-hostfxr-8.0|dotnet-templates-8.0 "Depends" on this, should be dropped from the seed.

> + * dotnet-hostfxr-8.0

dotnet8 -> dotnet-sdk-8.0 -> dotnet-runtime-8.0 "Depends" on this, should be dropped from the seed.

> + * dotnet-runtime-8.0

dotnet8 -> dotnet-sdk-8.0 "Depends" on this, should be dropped from the seed.

> + * dotnet-runtime-dbg-8.0

Note: This is not covered by Extra-Include, so it's fine to stay here.

> + * dotnet-sdk-8.0

This is a dependency of bin:dotnet8, can be dropped from the seed.

> + * dotnet-sdk-dbg-8.0

Note: This is not covered by Extra-Include, so it's fine to stay here.

> + * dotnet-targeting-pack-8.0

dotnet8 -> dotnet-sdk-8.0 "Depends" on this, should be dropped from the seed.

> + * dotnet-templates-8.0

dotnet8 -> dotnet-sdk-8.0 "Depends" on this, should be dropped from the seed.

> + * dotnet8

Note: This is the core of what we want :) Please add a comment, referencing the MIR bug here.

> + * netstandard-targeting-pack-2.1-8.0

dotnet8 -> dotnet-sdk-8.0 "Depends" on this, should be dropped from the seed.

> + * Extra-Exclude: dotnet-sdk-8.0-source-built-artifacts

IIUC this is not pulled in by anything (reverse Depends or Recommends). So no need to exclude it. Furthermore, The "Extra-Exclude" wouldn't work here, as it can only drop things from "Extra-Include" (see "supported" seed). Please drop this line.

$ reverse-depends -r noble dotnet-sdk-8.0-source-built-artifacts

So it can just be demoted by an AA after src:dotnet8 got promoted.



-- 
https://code.launchpad.net/~dviererbe/ubuntu-seeds/+git/ubuntu/+merge/463930
Your team Ubuntu Core Development Team is subscribed to branch ~ubuntu-core-dev/ubuntu-seeds/+git/ubuntu:noble.




More information about the Ubuntu-reviews mailing list