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

Issue on compile #207

Open
wpeckham opened this issue Apr 16, 2022 · 6 comments
Open

Issue on compile #207

wpeckham opened this issue Apr 16, 2022 · 6 comments

Comments

@wpeckham
Copy link

Following ARCH steps on Pinebook Pro (AARCH64) running Manjaro 22.04 the cmake steps seem to succeed, but the make step throws this log and fails. Are there alternate steps or additional instructions to bypass this issue?
RcloneBrowser/src/main_window.cpp:435:44: error: ‘QStringList QString::split(const QString&, QString::SplitBehavior, Qt::CaseSensitivity) const’ is deprecated: Use Qt::SplitBehavior variant instead [-Werror=deprecated-declarations]
435 | QStringList lines = version.split("\n", QString::SkipEmptyParts);
| ~~~~~~~~~~~~~^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
In file included from /usr/include/qt/QtCore/qobject.h:47,
from /usr/include/qt/QtCore/qabstractanimation.h:43,
from /usr/include/qt/QtCore/QtCore:6,
from /home/wpeckham/work/RcloneBrowser/src/pch.h:9,
from /home/wpeckham/work/RcloneBrowser/src/icon_cache.h:3,
from /home/wpeckham/work/RcloneBrowser/src/main_window.h:3,
from /home/wpeckham/work/RcloneBrowser/src/main_window.cpp:1:
/usr/include/qt/QtCore/qstring.h:609:17: note: declared here
609 | QStringList split(const QString &sep, SplitBehavior behavior,
| ^~~~~
/home/wpeckham/work/RcloneBrowser/src/main_window.cpp: In member function ‘void MainWindow::addStream(const QString&, const QString&)’:
/home/wpeckham/work/RcloneBrowser/src/main_window.cpp:1217:16: error: ‘void QProcess::start(const QString&, QIODevice::OpenMode)’ is deprecated: Use QProcess::start(const QString &program, const QStringList &arguments,OpenMode mode = ReadWrite) instead [-Werror=deprecated-declarations]
1217 | player->start(stream, QProcess::ReadOnly);
| ~~~~~~~~~~~~~^~~~~~~~~~~~~~~~~~~~~~~~~~~~
In file included from /usr/include/qt/QtCore/QtCore:170,
from /home/wpeckham/work/RcloneBrowser/src/pch.h:9,
from /home/wpeckham/work/RcloneBrowser/src/icon_cache.h:3,
from /home/wpeckham/work/RcloneBrowser/src/main_window.h:3,
from /home/wpeckham/work/RcloneBrowser/src/main_window.cpp:1:
/usr/include/qt/QtCore/qprocess.h:168:10: note: declared here
168 | void start(const QString &command, OpenMode mode = ReadWrite);
| ^~~~~
cc1plus: all warnings being treated as errors
make[2]: *** [src/CMakeFiles/rclone-browser.dir/build.make:230: src/CMakeFiles/rclone-browser.dir/main_window.cpp.o] Error 1
make[1]: *** [CMakeFiles/Makefile2:98: src/CMakeFiles/rclone-browser.dir/all] Error 2
make: *** [Makefile:136: all] Error 2

@wpeckham
Copy link
Author

Issue can be closed. There is a version in AUR that has been tuned to compile on Manjaro.

@spodhajecki
Copy link

I had the same errors on Ubuntu 22.04.3 LTS
in main_window.cpp, changed line 435 as follows:

// QStringList lines = version.split("\n", QString::SkipEmptyParts);
QStringList lines = version.split("\n", Qt::SkipEmptyParts);

Changed line 1217 as follows:

// player->start(stream, QProcess::ReadOnly);
player->setProgram(stream);
player->start(QProcess::ReadOnly);

@ccchan234
Copy link

same finding in latest chromebook's linux.
let me see if the above modify could help. thanks

@Inc44
Copy link

Inc44 commented Feb 25, 2024

The pull request has already been sent in July 2020 and is still awaiting approval: ce9cf52

@ccchan234
Copy link

hi everybody,
yeah the above modifications worked.
chatgpt suggest the same modifications.

ps, it's working on mine now. thanks all.

@sandy9707
Copy link

I had the same errors on Ubuntu 22.04.3 LTS in main_window.cpp, changed line 435 as follows:

// QStringList lines = version.split("\n", QString::SkipEmptyParts); QStringList lines = version.split("\n", Qt::SkipEmptyParts);

Changed line 1217 as follows:

// player->start(stream, QProcess::ReadOnly); player->setProgram(stream); player->start(QProcess::ReadOnly);

It's worked!

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

5 participants