Become a MacRumors Supporter for $50/year with no ads, ability to filter front page stories, and private forums.
Status
The first post of this thread is a WikiPost and can be edited by anyone with the appropiate permissions. Your edits will be public.

SnakeCoils

macrumors regular
Oct 17, 2018
133
60
Italy
Well, changing the declarations in the patch file (1002) about the OSX version did not change anything, same error during compile. For completeness the manual modification in ScreenSaver.cpp involved an additional change in the two lines as follow:

Code:
 m_okDisp = (IOPMAssertionCreate(kIOPMAssertionTypeNoDisplaySleep, kIOPMAssertionLevelOn, &m_idDisp) == kIOReturnSuccess);
 m_okSys  = (IOPMAssertionCreate(kIOPMAssertionTypeNoIdleSleep,  kIOPMAssertionLevelOn, &m_idSys)  == kIOReturnSuccess);

Otherwise the compiler would complain: for the above fix I have followed the suggestions in the main.log file.
About the patches not applied I noticed that while traces of 0001-0007 are found in the log file, the 1002 (that refers to Screensaver.cpp patch) never went considered during compile, I have tried to copy it in the first list of the Portfile patches (just after the 0007) but it did not worked :) Anyway the program runs great, a bit complicated at first sight but very customizable. Many thanks!
 
Last edited:

barracuda156

macrumors 68000
Original poster
Sep 3, 2021
1,757
1,285
Otherwise the compiler would complain: for the above fix I have followed the suggestions in the main.log file.
About the patches not applied I noticed that while traces of 0001-0007 are found in the log file, the 1002 (that refers to Screensaver.cpp patch) never went considered during compile

Thank you, I am being stupid, I put the patch into the wrong section of the port file. It has three different versions, so the patch simply was never applied in your build.

UPD. And it did not apply when you tried to add it, because that part is already been patched in patch 0003. Anyway, I will sort this out. Thanks for testing, it was very helpful.
 
  • Like
Reactions: SnakeCoils

barracuda156

macrumors 68000
Original poster
Sep 3, 2021
1,757
1,285
Well, changing the declarations in the patch file (1002) about the OSX version did not change anything, same error during compile. For completeness the manual modification in ScreenSaver.cpp involved an additional change in the two lines as follow:

Code:
 m_okDisp = (IOPMAssertionCreate(kIOPMAssertionTypeNoDisplaySleep, kIOPMAssertionLevelOn, &m_idDisp) == kIOReturnSuccess);
 m_okSys  = (IOPMAssertionCreate(kIOPMAssertionTypeNoIdleSleep,  kIOPMAssertionLevelOn, &m_idSys)  == kIOReturnSuccess);

Otherwise the compiler would complain: for the above fix I have followed the suggestions in the main.log file.
About the patches not applied I noticed that while traces of 0001-0007 are found in the log file, the 1002 (that refers to Screensaver.cpp patch) never went considered during compile, I have tried to copy it in the first list of the Portfile patches (just after the 0007) but it did not worked :) Anyway the program runs great, a bit complicated at first sight but very customizable. Many thanks!

I have finally rebased the PR with your change incorporated. Hopefully this gonna work now: https://github.com/macports/macports-ports/pull/23676
 
  • Like
Reactions: SnakeCoils

SnakeCoils

macrumors regular
Oct 17, 2018
133
60
Italy
Thank you very much, I am going to rebuild QMplay2 as soon as possible on my G4 MDD to confirm.

Update: Just tried to rebuild from your very latest GitHub repository but it failed. Apparently the ScreenSaver.cpp still have some issues in the same two lines but with different results.
Attached you will find the Log files from the build, the first before my tentative correction (just deleted the "WithName" from "IOPMAssertionCreateWithName" string) and the second after the correction, seems to dislike the parenthesis but here my programming knowledge stops :)
 

Attachments

  • main1.log.txt
    965 KB · Views: 6
  • main2.log.txt
    479 KB · Views: 8
Last edited:
  • Like
Reactions: barracuda156

barracuda156

macrumors 68000
Original poster
Sep 3, 2021
1,757
1,285
Consider trying these out:

lastfm-desktop https://ports.macports.org/port/lastfm-desktop/details (10.4 not supported at the moment, 10.5–10.6 should work)

QMMP, a music player with Qt-based GUI https://ports.macports.org/port/qmmp/details

albafetch, a faster neofetch replacement written in C https://ports.macports.org/port/albafetch/details (on 10A190 you will see a funny error on launch, that is because 10A190 lacks ppc slices in a few SystemProfiler components, but those can be borrowed from 10.5.8).
 
Register on MacRumors! This sidebar will go away, and you'll see fewer ads.