-
Notifications
You must be signed in to change notification settings - Fork 135
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
fixed compile errors and segfaults with gcc 14.x #192
Conversation
Yes!! previously getting this issue (in Fedora) : #191 Seems resolved I do get these warnings:
|
Same here, assumed the warnings were less important for now. This takes the first step in getting it working (I'm running it right now, in fact). Based on the commit history, it looks like this code has not been maintained for some time, so I expect there are a number of things still to do. (For the same reason, I'm actually surprised to see a comment on this PR. 😅 ) Anyhow, now that I know there are other users maybe I'll do a little more (e.g., perhaps create an Arch package for AUR), but going to keep this PR limited to "making it work" for now. |
I tried to setup a headless machine, instead I setup some awful auto-login GUI- bash gnome-autostart thing. |
Marked as WIP. startcrypto is broken, need to fix. |
@lneely There is the AUR package https://aur.archlinux.org/packages/pcloudcc-git that I was maintaining, tracking the main branch here. Do you want me to add you as maintainer in order to track your commits? I am not very invested in the package anymore, as I am using pcloud very sparingly, but the cli is much better than the overbloated electron app, so an active maintainer would be great. |
@coxackie Beyond this I didn't plan on doing much else, but what the hell; it will give me something to do, and I wouldn't mind helping out. Please bear with me as I learn, because I've never actually maintained an AUR package or any package for that matter. |
Confirmed, this is compiling better for me |
It's a process, was more broken than I thought it was. Starting to feel some ass pain. |
Fixed the original segfault I was hunting, was apparently caused by writing outside some array boundaries. Now hunting another one. |
OK! fixed the segfaults I found. @asdf23, also resolved some additional compilation warnings. I think this is ready for merge. |
@coxackie let me know if you still want a maintainer for the AUR package. If this repo isn't active anymore, and I suspect it may not be, I might just maintain the code independently. |
Confirming that lneely:master branch builds, runs and works on Manjaro. Having this in AUR package would be great. |
@coxackie I just registered, my username is lkn. |
To all interested: it has now been over 2 weeks, and this PR seem to have fallen into the same void as the 14 other open PRs (some of which are years old). Despite this, I will keep the PR open in case this repo ever sees any more activity. At the same time, I've created my own independent fork of console-client with a boring name because I'm lazy. I promise to maintain it as inconsistently and haphazardly as I do all the other code I don't get paid to work on. |
I don't know where to post this. This tool makes pcloud 10 times more valuable. PCloud should be paying you for this. That company has really lost there way. It's a shame. That password mgmt tool they came out with recently is garbage. There's a major bug in it, they are not addressing it, not going to address it. I do have a contact over there, I'll try to reach out to them. |
@asdf23 🍻 I already get paid to code, just not by pcloud. I'm just happy that the client is free software so this is possible to do. Not many companies do this, and it was a big reason I chose pcloud. |
@lneely that's not at all surprising :) But at issue is pCloud taking responsibility for their code. For instance if they introduce a new feature (like 2FA) this thing all goes down, and I doubt they'd even be aware of the impact. What's even more ridiculous is that this tool could be it's own feature - a bullet point on the list of why pCloud is better than the various competitors. Instead we are seeing their "values in action". arghh |
@lneely your own fork with its own name sounds like a plan, pcloud team can always come back and merge back your code into theirs. I'm a software developer myself -- can you setup a github sponsorship or any other way to chip in small monthly amounts to express appreciation of your spare-time work on this? Keeping with the compilation errors, I just got a new toy, ClockworkPi uConsole which is basically a device that turns Raspberry Pi CM4 into a laptop. The operating system that came with it is Raspberry OS based on Debian 11 with kernel 5.10.17 and gcc 10.2.1, all in all pretty ancient. I've built your fork successfully on it and it works (thank you once again), but I before getting there the make was failing with the error message |
@tomash much appreciated, really, but I would have to check the German immigration laws before I could do anything like that. Depends, for example, if it counts as a form of "self employment," which is not currently allowed under the terms of my residence permit. I will copy and paste what you've reported here into my fork and tag you there so I can more easily track it. While I don't have a raspberry pi myself, perhaps I can simulate the environment. Right now I'm in "seek and destroy" mode to make my job easier. This will likely take some time, but happy to try and look into it whenever I can. There might be a decent systematic way to support that case. (Trying to avoid special cases like the plague) 🍻 |
Closing due to maintainer inactivity. |
If @pcloudcom isn't going to maintain the software it constitutes a misleading representation of the product's capabilities. Which could potentially constitute false advertising and deceptive trade practices. I wonder if pCloud could be bothered to edit the HTML with the misleading claim. Or perhaps we can find someone here on github that would update the HTML for them. |
The pcloud console client was not compiling on my system due to multiple compiler errors (e.g., type mismatches without casts). My system is running Arch, and my gcc version is 14.1.1 20240522.
This PR fixes the compilation errors, and cleans up some unused variables & non-existent headers. I also added some things to .gitignore (e.g., compile_commands.json generated by bear, clangd .cache, and CMake-generate files).