#1 remove strstream in ktview.cpp

Merged
MicheleC merged 2 commits from gg-tdeadmin-kcron into master 1 year ago
cethyel commented 1 year ago

I’m trying to convert tdeadmin autotool to cmake building system, in the process I’ve run into some deprecated such as strstream in ktview.cpp; strstream has been deprecated since C++98, therefore I’d like to get rid of it.

I'm trying to convert tdeadmin autotool to cmake building system, in the process I've run into some deprecated such as strstream in ktview.cpp; strstream has been deprecated since C++98, therefore I'd like to get rid of it.
MicheleC commented 1 year ago
Owner

Hi Gregory, Thanks for the various PR. Just for your info, we are waiting before merging any PR until R14.0.5 has been built on main server. This to make sure the builders can fully focus on building R14.0.5 without triggerring unnecessary R14.1.x builds. This hopefully will take only few days more. Keep up the great work.

Hi Gregory, Thanks for the various PR. Just for your info, we are waiting before merging any PR until R14.0.5 has been built on main server. This to make sure the builders can fully focus on building R14.0.5 without triggerring unnecessary R14.1.x builds. This hopefully will take only few days more. Keep up the great work.
SlavekB commented 1 year ago
Owner

I can confirm that the builds for final R14.0.5 are moving forward, but a few days will still be needed.

I can confirm that the builds for final R14.0.5 are moving forward, but a few days will still be needed.
MicheleC commented 1 year ago
Owner

Patch from Gregory is good and can be merged. Since sstream is now used by default, a check for sstream during configure is no longer necessary (commit added).

Patch from Gregory is good and can be merged. Since sstream is now used by default, a check for sstream during configure is no longer necessary (commit added).
MicheleC commented 1 year ago
Owner

Merged and pushed upstream.

Merged and pushed upstream.
MicheleC deleted branch gg-tdeadmin-kcron 1 year ago
MicheleC added this to the R14.0.6 release milestone 1 year ago
The pull request has been merged.
Sign in to join this conversation.
Loading…
Cancel
Save
There is no content yet.