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

Meson Build System Merge #548

Open
wants to merge 26 commits into
base: master
Choose a base branch
from
Open

Meson Build System Merge #548

wants to merge 26 commits into from

Conversation

DMJC
Copy link

@DMJC DMJC commented Apr 6, 2024

Implemented Meson Build script. Currently only tested on linux

Copy link
Author

@DMJC DMJC left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Meson.build is the only file that needs to be merged initially. In my repository I've merged all the patches that were in the patch folder. Otherwise you can merge my whole repository which will clean out most of the auto-tools configs.

@DMJC
Copy link
Author

DMJC commented Apr 6, 2024

I'm expecting this PR to get knocked back, if you have guidance on how you want this merged I'll try to work with you on it.

@DMJC DMJC marked this pull request as ready for review April 6, 2024 10:05
@EionRobb
Copy link
Collaborator

EionRobb commented Apr 6, 2024

Do you think it would be possible to bring through the original hg commit history from the "missing files" in the pidgin source?

Copy link
Collaborator

@EionRobb EionRobb left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks good 🙂

Have you tried building a Windows dll, out of interest? ignore me, reread the pr comment 😄

meson.build Outdated Show resolved Hide resolved
@DMJC
Copy link
Author

DMJC commented Sep 5, 2024

Is there anyone with write access who can merge this?

@grimmy
Copy link
Collaborator

grimmy commented Sep 5, 2024

I'll try to take a look tonight.

@DMJC
Copy link
Author

DMJC commented Sep 15, 2024

I have also implemented 2FA for Facebook.

@@ -4,4 +4,16 @@ this plugin is back-ported for purple2.

This project is not affiliated with Facebook, Inc.

This has been forked from https://github.com/dequis/purple-facebook/ - Thanks Dequis for Maintaining this project for so long.
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This probably shouldn't be merged with this here, it'd be very confusing.

Copy link
Contributor

@jaymzh jaymzh left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

The patches were there, I think, so that it was easy to pull in the upstream bitlbee repos, as those were all patches on top. I think merging them will prevent that....

@DMJC
Copy link
Author

DMJC commented Sep 17, 2024 via email

@jaymzh
Copy link
Contributor

jaymzh commented Sep 17, 2024

You can't merge in bitlbee directly, the libpurple and bitlbee apis have clear differences that prevent merging. I'll clean up my readme.

I may be confusing different things here, so bear with me, but my understanding is the reason there's a patches directory is because it's checked out from mercurial and patched to work with libpurple. It's described a bit here:

#516

@DMJC
Copy link
Author

DMJC commented Sep 17, 2024

I've just looked at what it's doing. It's actually using Mercurial to pull code from the Pidgin repository. It's not downloading anything from Bitlbee. The URL: https://keep.imfreedom.org/pidgin/pidgin is the pidgin main repo. The "patches" are the actual Facebook plugin. So we're checking out the pidgin repository from 2017!, then applying patches to create our plugin. This is nuts. This is the patch revision: https://keep.imfreedom.org/pidgin/pidgin/rev/9ff9acf9fa14. This system needs to go in the bin, the more you look the worse it gets, there are even libpurple fixes in upstream that never get merged in with the build scripts. I will see what I can do.

@DMJC
Copy link
Author

DMJC commented Sep 17, 2024

I've found the last revision ever uploaded to Mercurial. It's the revision we've been using. There are no more revisions past that and they actually nuked the plugin from the repository. Check that out as a hg repository and convert it to git, upload to github to preserve commit history, then we need to delete everything that we don't use (all the purple stuff except our plugin), and then we have to go through our patches and verify that they apply, and commit each patch so we have a new history of changes. This is going to take a while.

@DMJC
Copy link
Author

DMJC commented Sep 17, 2024

I've created a new repository called https://github.com/DMJC/purple-fb-master I've attempted to extract and import the Mercurial history for pidgin into it. Once the upload finishes I will try to clean up the repository to match The Facebook Plugin.

@DMJC
Copy link
Author

DMJC commented Sep 17, 2024

The revision of the code that the facebook plugin checks out/builds against is: https://keep.imfreedom.org/pidgin/pidgin/rev/9ff9acf9fa14 if you look in the update.sh file in the purple-facebook project, it calls a MANIFEST file and a VERSION file. The MANIFEST is the files it grabs from Mercurial, the VERSION is the revision it checks out, and there is also a MANIFEST_VOIDS which just creates empty files in includes file the project. Most of the pidgin project code/files is not needed. I need some advice on what we should clean up/keep in terms of history.

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

Successfully merging this pull request may close these issues.

4 participants