Skip to content

Conversation

@sherlockcoin
Copy link
Contributor

@sherlockcoin sherlockcoin commented Jun 6, 2019

Background image of the OSX install window is mis-sized #379

Background image of the OSX install window is mis-sized navcoin#379
@aguycalled
Copy link
Member

@sherlockcoin would it be possible to automate the version which appears on the background? otherwise i think i would rather remove it and leave just NAVCOIN CORE

@sherlockcoin
Copy link
Contributor Author

Sure! it is possible, just wanted to make sure people knew the right version when downloading the DMG.

Dynamic Labling
* Dynamic labeling
@mxaddict
Copy link
Contributor

mxaddict commented Jun 6, 2019

utACK

@mxaddict
Copy link
Contributor

mxaddict commented Jun 6, 2019

I think these changes will require testing with gitian build

@sherlockcoin
Copy link
Contributor Author

tACK
QT GUI compiled and tested on OS X 10.12.6 (Sierra) - Non-deterministic.

@aguycalled
Copy link
Member

gitian fails

+ make osx_volname
echo NavCoin-Core >osx_volname
+ make deploydir
sed 's/PACKAGE_NAME/NavCoin Core/' < "contrib/macdeploy/background.svg" |  -f png -d 36 -p 36 | /usr/bin/convert - dpi36.background.tiff
/bin/bash: -f: command not found
convert: no decode delegate for this image format `' @ error/constitute.c/ReadImage/504.
convert: no images defined `dpi36.background.tiff' @ error/convert.c/ConvertImageCommand/3258.
make: *** [dpi36.background.tiff] Error 1
Makefile:1195: recipe for target 'dpi36.background.tiff' failed

@sherlockcoin
Copy link
Contributor Author

Can you tell me the gitian system and dependencies please? Looks like an imagemagick error.

@sherlockcoin
Copy link
Contributor Author

Found it! :) One moment.

@sherlockcoin
Copy link
Contributor Author

@aguycalled is it possible to verify if its an imagemagick issue? Would it possible to reinstall imagemagick and re-test?

@aguycalled
Copy link
Member

aguycalled commented Jun 7, 2019 via email

@sherlockcoin
Copy link
Contributor Author

Indeed. I will do the needful. 👍

Gitian build fails Fix.
Python usage Fix
DMG Mounting Crash Fix.
@sherlockcoin
Copy link
Contributor Author

Screen Shot 2019-06-09 at 10 34 06 PM

Gitian Build success. :)

Copy link
Contributor

@mxaddict mxaddict left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

testedACK

I've compiled and tested on Ubuntu 18.04 (Just for sanity check)

I can't test on OSX myself.

@sherlockcoin
Copy link
Contributor Author

@mxaddict Thank you mate. It was quite a fix lol. 😄

@mxaddict
Copy link
Contributor

@sherlockcoin the DMG looks nice and professional! Nice work! 👍

@sherlockcoin
Copy link
Contributor Author

@sherlockcoin the DMG looks nice and professional! Nice work! 👍

Thank you mate. <3 Cheers! :)

@mxaddict
Copy link
Contributor

BTW, I'm doing a gitian build for OSX now, and will ask @lordhace to check my build, will post results here 🥇

@sherlockcoin
Copy link
Contributor Author

Can @aguycalled and everyone review and merge this? :)

@lordhace
Copy link
Contributor

BTW, I'm doing a gitian build for OSX now, and will ask @lordhace to check my build, will post results here 🥇

Works on OSX 10.13.6
navMacOS-1

Copy link
Member

@aguycalled aguycalled left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Tested

@aguycalled aguycalled merged commit e393fe4 into navcoin:master Jun 11, 2019
@mxaddict
Copy link
Contributor

mxaddict commented Jun 11, 2019 via email

@sherlockcoin
Copy link
Contributor Author

Awesome! :)

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants