Modify

#23335 closed defect (fixed)

MacOS.zip does not build

Reported by: stoecker Owned by: team
Priority: normal Milestone:
Component: Installer MacOS Version:
Keywords: Cc: Don-vip, taylor.smock

Description

Attachments (0)

Change History (8)

comment:1 by taylor.smock, 14 months ago

It looks like it is due to signjar failing.

Looking at the job outputs (see https://github.com/JOSM/josm/actions/runs/7091380242/job/19331552037 )

dist:
     [echo] Revision 18905
      [jar] Building jar: /Users/runner/work/josm/josm/dist/josm-custom.jar
  [signjar] Signing JAR: /Users/runner/work/josm/josm/dist/josm-custom.jar to /Users/runner/work/josm/josm/dist/josm-custom.jar as 
  [signjar] jarsigner error: java.lang.RuntimeException: keystore load: keystore password was incorrect

EDIT: Relevant code from build.xml:

        <!-- Sign jar if all environment variables are set -->
        <signjar jar="${dist.jar}" alias="${env.SIGN_ALIAS}" tsaurl="${env.SIGN_TSA}"
            keystore="${env.SIGN_KEYSTORE}" storepass="${env.SIGN_STOREPASS}" keypass="${env.SIGN_KEYPASS}" if:set="sign.jar" />

We probably need to update env variables on GitHub.

Last edited 14 months ago by taylor.smock (previous) (diff)

comment:2 by stoecker, 14 months ago

Probably I added wrong password (can't check, GitHub secrets are write only). Replaced them again and triggered rebuild.

comment:3 by stoecker, 14 months ago

Ticket #23336 has been marked as a duplicate of this ticket.

comment:4 by stoecker, 14 months ago

Hmm. Password seems correct now (last letter was missing :-), but it seems there are secret secret variables (i.e. I didn't get displayed that they exist), so I didn't update them.

comment:5 by taylor.smock, 14 months ago

Assuming we didn't change the keystore password (I think this would be SIGN_STOREPASS), it should be changeit, IIRC.

comment:6 by stoecker, 14 months ago

In 18906/josm:

add missing APPLE_ID, see #23335

in reply to:  5 comment:7 by stoecker, 14 months ago

Replying to taylor.smock:

Assuming we didn't change the keystore password (I think this would be SIGN_STOREPASS), it should be changeit, IIRC.

Assumption wrong ;-) When I make new keys I also use new passwords...

And these are done with this command

alias mkpasswd='perl -e "\$_=qx(dd if=/dev/urandom count=1 2>/dev/null);s/[^A-Za-z0-9]//g;\$n=\$ARGV[0]||15;/(.{\$n})/;print qq(Your new password: \$1\n)"'

which outputs a proper 15 letter alphanumeric password.

Note: That command is meant to be short. It wastes a lot of randomness. That's ok as long as it's used seldom!

Last edited 14 months ago by stoecker (previous) (diff)

comment:8 by stoecker, 14 months ago

Resolution: fixed
Status: newclosed

Modify Ticket

Change Properties
Set your email in Preferences
Action
as closed The owner will remain team.
as The resolution will be set.
The resolution will be deleted. Next status will be 'reopened'.

Add Comment


E-mail address and name can be saved in the Preferences .
 
Note: See TracTickets for help on using tickets.