Bug 346916 - changes to make it compile on Mac
Summary: changes to make it compile on Mac
Status: RESOLVED UNMAINTAINED
Alias: None
Product: kreenshot-editor
Classification: Applications
Component: general (show other bugs)
Version: unspecified
Platform: Other Linux
: NOR normal
Target Milestone: ---
Assignee: Gregor Mi
URL:
Keywords:
Depends on:
Blocks:
 
Reported: 2015-04-29 18:59 UTC by Gregor Mi
Modified: 2020-11-24 23:58 UTC (History)
2 users (show)

See Also:
Latest Commit:
Version Fixed In:


Attachments

Note You need to log in before you can comment on or make changes to this bug.
Description Gregor Mi 2015-04-29 18:59:49 UTC
By diego via email:

I have tried your app on a (stupid) mac, and I can tell you it compiles. With some teaks:

1.    The shell script to create the clean symlinks is really broken.
      1.  Please use "set -x" at the top
      2.  The usage of "--force" is not available on macs. Look for other "--" usage on the script.
2.    In src/lib/kreenshoteditor/ui/selectionhandles/selectionhandlesmgr.cpp I needed to include "math.h", otherwise "floor()" is not available to me.

Then in runtime, the 4th test fails (make test).
Comment 1 Gregor Mi 2015-04-29 19:03:52 UTC
What would the "set -x" do? If I do it on the console it prints variables.
Comment 2 Gregor Mi 2015-06-21 11:25:39 UTC
ah, it's for debugging: http://tldp.org/LDP/Bash-Beginners-Guide/html/sect_02_03.html
Comment 3 Justin Zobel 2020-11-24 05:04:24 UTC
Can you please test and confirm if this issue is still occurring or if this bug report can be marked as resolved. I've set the bug status to "needsinfo" pending your response, please change back to "reported" or "resolved" when you respond, thanks.
Comment 4 Gregor Mi 2020-11-24 16:43:28 UTC
Thanks for looking after the state of these issues.
I am the original author of kreenshot-editor and the project is no longer maintained by me. And I am not aware of any further contributions, except those from 2015: https://github.com/KDE/kreenshot-editor. The aim of the project is probably better implemented by https://github.com/ksnip/kImageAnnotator.
Comment 5 Justin Zobel 2020-11-24 23:58:31 UTC
Thanks for the update Gregory, I'll mark this resolved/unmaintained.