Giter Club home page Giter Club logo

Comments (18)

zrose584 avatar zrose584 commented on September 23, 2024 1

It seems like official patches got released yesterday, here the changelog.
Yet nothing on salsa, but probably won't take too long..

from ungoogled-chromium-debian.

Eloston avatar Eloston commented on September 23, 2024 1

Done as of 94a07d5

from ungoogled-chromium-debian.

Eloston avatar Eloston commented on September 23, 2024

Yeah, I am not too happy about this either. I will look into it, but it will take me some time.

I will try to keep things as Debian-like as possible, but I will probably take some shortcuts (e.g. use some more bundled libraries instead of system libraries) since I don't have that much time to patch these kinds of dependencies.

from ungoogled-chromium-debian.

zrose584 avatar zrose584 commented on September 23, 2024

Hi, I also tried to port the patches to chromium 78:
https://github.com/zrose584/ungoogled-chromium-debian/tree/zrose584_deb10_78

But currently I have no idea if they work.. it starts building though

from ungoogled-chromium-debian.

zrose584 avatar zrose584 commented on September 23, 2024

@nurupo Make sure your debian/patches/series file didn't changed.
The only change I made was to remove gcc6/atomic.patch, as this patch is obsolete now.

edit:
I think we can't use use_system_harfbuzz=true anymore, since the required harfbuzz-subset is not in any debian repositorys yet.

from ungoogled-chromium-debian.

nurupo avatar nurupo commented on September 23, 2024

@zrose584 thank you for the concern but I have deleted all my files/patches. No point in keeping them if I can't get it to work.

from ungoogled-chromium-debian.

nurupo avatar nurupo commented on September 23, 2024

If I remember right, the gcc6/atomic.patch you have mentioned was already applied in the upstream.

I believe I had one more obsolete patch though. There was a patch that patched only no longer existing files. I didn't bother checking git to see if the files were renamed/moved so I excluded that patch too.

from ungoogled-chromium-debian.

tangalbert919 avatar tangalbert919 commented on September 23, 2024

Knowing Debian, they probably won't update their salsa until they have Chromium 78 in their testing or stable branches, so it could take a while.

In the meantime, I could just update the patches in the Ubuntu branches right now, since Ubuntu has already released their patches for that version in their package archive.

from ungoogled-chromium-debian.

zrose584 avatar zrose584 commented on September 23, 2024

FIY I integrated the official debian patches over on my fork.
I had to edit some patches, and even generate new ones to make it work.
There are also many duplicate patches now..

You can see exactly what commands I used over on OBS, since thats what I used to build it: link.

I had to use clang-7 since OBS does not provide backported packages for buster. Luckily it just worked anyways.
I now wonder.. what is the reason of using clang instead of gcc? The debian patches are specifically made for gcc, and potentially even break compilation with clang.

Btw, I partially used OBS in hope of showing @Eloston a way of automating the debian builds, since that seems to be discussed for more than 3 years now.

from ungoogled-chromium-debian.

Eloston avatar Eloston commented on September 23, 2024

I now wonder.. what is the reason of using clang instead of gcc? The debian patches are specifically made for gcc, and potentially even break compilation with clang.

There's no strong reason. When I started ungoogled-chromium, I thought it made more sense to use the compiler that Chromium officially uses, since I thought it would produce more efficient builds and require fewer changes to Chromium. As of now, I don't know how much GCC vs LLVM has on efficiency, but it does eliminate the need for GCC patches. If there is a strong reason to, we could change it back to GCC.

from ungoogled-chromium-debian.

tangalbert919 avatar tangalbert919 commented on September 23, 2024

The patches have been updated. Now it's just a matter of finding out if it builds.

EDIT: Thanks, @Eloston.

from ungoogled-chromium-debian.

Eloston avatar Eloston commented on September 23, 2024

@zrose584 It's interesting to see that OBS builds Chromium so quickly. It looks promising; could you post your findings to ungoogled-software/ungoogled-chromium#17? I'm also wondering if there is a way to add packages from backports (or apt.llvm.org).

@tangalbert919

The patches have been updated. Now it's just a matter of finding out if it builds.

Right now they don't. Trying out some changes before I push again.

from ungoogled-chromium-debian.

nurupo avatar nurupo commented on September 23, 2024

Salsa repo got updated a couple days ago. Also, Debian's Chromium includes the VAAPI patch now.

from ungoogled-chromium-debian.

zrose584 avatar zrose584 commented on September 23, 2024

What I found was that the chromium_78.0.3904.87.orig.tar.xz (250MB) has many files removed compared to chromium-78.0.3904.87.tar.xz (740MB).
Deleting these files did not only reduce compile time, but also made some unoffical patches unnecesary which I had to make and use before.
But most importantly, after removing the files, the chromium build stopped crashing on shutdown. I applied no further patches, just removed files.
For this I added make_lite.py. I created make_lite_list.txt by diffing the directories.
I suggest you add this to master.

from ungoogled-chromium-debian.

tangalbert919 avatar tangalbert919 commented on September 23, 2024

Right now, I've got some changes currently in the works to make debian_buster buildable while also addressing the unknown warning spam problem. I'll probably have those changes ready by the end of today.

from ungoogled-chromium-debian.

Eloston avatar Eloston commented on September 23, 2024

Update: I pushed some new changes to debian_buster. I still need to confirm that the new changes are enough for a working build.

from ungoogled-chromium-debian.

Eloston avatar Eloston commented on September 23, 2024

I should note a few new changes in the new commits:

  • Added debian/scripts/remove_copyright_excluded and debian/devutils/fix_copyright_excludes.py to enforce deletion of files based on debian/copyright. This should also help fix creating a valid .orig.tar.xz when building from a source package.
  • Added debian/scripts/{apply,revert}_domainsubstitution to make it easier to apply/revert domain substitution after a build failure (instructions documented in the README)
  • A few patches were added/removed. See commit log for details.

from ungoogled-chromium-debian.

tangalbert919 avatar tangalbert919 commented on September 23, 2024

I am currently testing to see if these changes are enough for a working build right now in a Virtual Machine. This will take an hour or two.

from ungoogled-chromium-debian.

Related Issues (20)

Recommend Projects

  • React photo React

    A declarative, efficient, and flexible JavaScript library for building user interfaces.

  • Vue.js photo Vue.js

    🖖 Vue.js is a progressive, incrementally-adoptable JavaScript framework for building UI on the web.

  • Typescript photo Typescript

    TypeScript is a superset of JavaScript that compiles to clean JavaScript output.

  • TensorFlow photo TensorFlow

    An Open Source Machine Learning Framework for Everyone

  • Django photo Django

    The Web framework for perfectionists with deadlines.

  • D3 photo D3

    Bring data to life with SVG, Canvas and HTML. 📊📈🎉

Recommend Topics

  • javascript

    JavaScript (JS) is a lightweight interpreted programming language with first-class functions.

  • web

    Some thing interesting about web. New door for the world.

  • server

    A server is a program made to process requests and deliver data to clients.

  • Machine learning

    Machine learning is a way of modeling and interpreting data that allows a piece of software to respond intelligently.

  • Game

    Some thing interesting about game, make everyone happy.

Recommend Org

  • Facebook photo Facebook

    We are working to build community through open source technology. NB: members must have two-factor auth.

  • Microsoft photo Microsoft

    Open source projects and samples from Microsoft.

  • Google photo Google

    Google ❤️ Open Source for everyone.

  • D3 photo D3

    Data-Driven Documents codes.