Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

rename remaining files to Autorgam #124

Conversation

celuchmarek
Copy link
Member

Súbory v src/main/scripts/resources/mac-launcher/ ešte bude treba asi nejako pregenerovať/editovať na Macu. Sú to nejaké binárky a na linuxe s nimi nemôžem nič.

@celuchmarek celuchmarek changed the title rename remaning files to Autorgam rename remaining files to Autorgam Apr 20, 2023
@celuchmarek
Copy link
Member Author

@pomali alebo @alhafoudh , vy ste Mac users. Nevedeli by ste poradiť, čo s týmto spraviť, prípadne niečo skompilovať?

Domnievam sa, že v tom adresári sú binárky potrebné na spúšťanie na Macu cez launcher. Menili sme názov z Octosign na Autogram a aj protokol zo signer:// na autogram://. V týchto binárkach sú pravdepodobne staré názvy a asi to treba zmeniť.

Tiež by sme potrebovali otestovať inštaláciu na Macu a potom aj volanie zo safari, ktoré vraj povoľuje iba https. Ale to až s releasom buildeným s týmto fixom.

@alhafoudh
Copy link
Contributor

@celuchmarek Pozrel som to, v tom main.scpt je toto:

# Launches the app with the custom protocol URL
# Desktop.setOpenURIHandler was not called on the first launch
# See https://bugs.openjdk.java.net/browse/JDK-8198549
on open location this_URL
	set appPath to (POSIX path of (path to me)) & "Contents/MacOS/OctosignApp"
	set appScript to (appPath & " --url=" & this_URL)
	do shell script appScript & " > /dev/null 2>&1 &"
end open location

Ako to upravim?

Potom ten MacOS/Octosign to je zkompilovana binarka, ta by podla mna mala byt v .gitignore. Pozrel tom lepsie v cez tyzden.

@celuchmarek
Copy link
Member Author

# Launches the app with the custom protocol URL
# Desktop.setOpenURIHandler was not called on the first launch
# See https://bugs.openjdk.java.net/browse/JDK-8198549
on open location this_URL
	set appPath to (POSIX path of (path to me)) & "Contents/MacOS/AutogramApp"
	set appScript to (appPath & " --url=" & this_URL)
	do shell script appScript & " > /dev/null 2>&1 &"
end open location

@alhafoudh , vďaka. Iba OctosignApp na AutogramApp.

A v src/main/scripts/resources/mac-launcher/Resources/Autogram.rsrc nevieš, čo je? Ja som ho zatiaľ prenenoval, ale tiež je to binárka.

@alhafoudh
Copy link
Contributor

@celuchmarek pushol som tuto upravu.
Co sa tyka Autogram.rsrc, tam podla vsetkeho by mali byt ikonky a taketo assety appky, ale nikto som nenasiel referenciu ze kto to vlastne kedy pouziva. Je ale pravda ze moc casu som tomu nedal.

@celuchmarek celuchmarek marked this pull request as ready for review April 22, 2023 21:49
@celuchmarek
Copy link
Member Author

Super, vďaka 🙂

@celuchmarek
Copy link
Member Author

@jsuchal máme od Ahmeda zmenený ten applescript. Je to v target branchi, lebo tento PR robíme vlastne z iného remote. Každopádne, takto by už malo byť všetko build related premenované.

@jsuchal jsuchal merged commit fc6c172 into slovensko-digital:refactor-spike-2022 Apr 23, 2023
@jsuchal jsuchal deleted the GO-74/rebrand-all-resources-to-autogram branch April 23, 2023 06:44
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants