Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Deprecate cef.redist packages #93

Closed
3 tasks done
amaitland opened this issue Feb 4, 2021 · 14 comments
Closed
3 tasks done

Deprecate cef.redist packages #93

amaitland opened this issue Feb 4, 2021 · 14 comments
Assignees

Comments

@amaitland
Copy link
Member

amaitland commented Feb 4, 2021

  • Add extra copy of locales to chromiumembeddedframework.runtime arch specific packages
  • Move Copy targets into chromiumembeddedframework.runtime arch specific packages
  • chromiumembeddedframework.runtime package would need to add support for the build folder (currently only uses buildTransitive)

Target is to make them compatible so we can deprecate the old Cef.redist packages.

@amaitland amaitland self-assigned this Feb 4, 2021
@amaitland amaitland changed the title Investigate depreciating cef.redist packages Investigate deprecating cef.redist packages Feb 5, 2021
amaitland added a commit that referenced this issue Jul 30, 2021
…older

previously only buildTransitive was supported.

Issue #93
amaitland added a commit that referenced this issue Jul 30, 2021
…hromiumembeddedframework.runtime packages

Haven't tested to see if folder structure is preserved, still need to do that.

Issue #93
@amaitland
Copy link
Member Author

Commit 8a902f2 adds the required props.

Haven't tested to see if the folder structure for the swiftshader/locales directories is preserved.

amaitland added a commit that referenced this issue Jul 30, 2021
…hromiumembeddedframework.runtime packages

Haven't tested to see if folder structure is preserved, still need to do that.

Issue #93
@amaitland
Copy link
Member Author

There was a problem with the net core package when using a runtime identifier, the locales/swiftshader files weren't being copied. Revert changes for now.

@amaitland
Copy link
Member Author

It doesn't look like moving the locales/swiftshader into the arch packages is possible for the net core packages. It may still be possible though not worth the effort.

@kpreisser
Copy link
Contributor

Hi @amaitland,
after upgrading to CefSharp.Common.NETCore 93.1.70-CI4201 (with chromiumembeddedframework.runtime 93.1.11), we face the issue that the locales and swiftshader folders are no longer copied when using a RuntimeIdentifier like win-x64, and thus resources fail to load. Can the changes for this be reverted/fixed?

Thanks!

@amaitland
Copy link
Member Author

@kpreisser Thanks for the reminder. The changes will need to be reverted as per my comment above.

@amaitland
Copy link
Member Author

If you remove the packages from your cache and redownload them the problem should be resolved.

build https://ci.appveyor.com/project/cefsharp/cef-binary/builds/40634148 using https://github.com/cefsharp/cef-binary/tree/cef/93

I've left master unchanged as I'll test replacing the runtime.json with a .props/.targets file that adds the PackageReference entries based on RuntimeIdentifier

Reference https://github.com/cefsharp/CefSharp/blob/master/NuGet/PackageReference/CefSharp.Common.NETCore.targets#L142

If that doesn't work then it's likely this idea won't work.

@kpreisser
Copy link
Contributor

If you remove the packages from your cache and redownload them the problem should be resolved.

build https://ci.appveyor.com/project/cefsharp/cef-binary/builds/40634148 using https://github.com/cefsharp/cef-binary/tree/cef/93

Thanks! Using these new packages seems to work.

@amaitland
Copy link
Member Author

It appears that swiftshader is being replaced and at some point in the future we can stop shipping the swiftshader folder as the new dlls are included in the base directory. This will simplify our packages slightly and potentially make deprecating cef.redist easier as we only have to deal with the locales, which are likely duplicated between the different archs currently (haven't confirmed this, will need to hash them all and confirm the files are the same across the three archs)

Reference https://bitbucket.org/chromiumembedded/cef/issues/3176/win-linux-ship-vk_swiftshaderdll-vulkan

@amaitland
Copy link
Member Author

Issue https://bugs.chromium.org/p/chromium/issues/detail?id=1060139 tracks removing of Swiftshader.

Once Swiftshader has been removed moving locales into a new package should be fairly trivial. I haven't checked though I suspect that locales are the same for each arch at least the windows ones, so we can possibly just ship a single set of files.

If we only need to ship a single set of locales then it might be possible to use contentfiles to include the relevant files and programmatically set CefSettings.LocalesDirPath. Haven't done any testing around this yet.

@amaitland
Copy link
Member Author

Removing of SwiftShader has started https://bugs.chromium.org/p/chromium/issues/detail?id=1060139#c124

@amaitland
Copy link
Member Author

@amaitland amaitland changed the title Investigate deprecating cef.redist packages Deprecate cef.redist packages Dec 27, 2023
amaitland added a commit that referenced this issue Jan 28, 2024
amaitland added a commit that referenced this issue Feb 4, 2024
* Remove cef.redist packages
* Change props names to match convention set in runtime package
* Add Copy targets for Locales and fix naming

Issue #93
amaitland added a commit that referenced this issue Feb 4, 2024
amaitland added a commit that referenced this issue Feb 4, 2024
@amaitland
Copy link
Member Author

We've exceeded the quote limits on MyGet so the old cef.redist packages will no longer be generated going forward.

They newer chromiumembeddedframework.runtime packages will be used in CefSharp. Anyone using the cef.redist pacakges directly will need to migrate as well.

@amaitland
Copy link
Member Author

@amaitland
Copy link
Member Author

cef.redist.x64 => chromiumembeddedframework.runtime.win-x64
cef.redist.x86 => chromiumembeddedframework.runtime.win-x86

The packages have now been marked as deprecated on Nuget.org.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants