Skip to content

Comments

Mise à jour du script pour Screaming Frog SEO avec screaming-frog-seo…#1900

Closed
KarimAltavia wants to merge 4 commits intoInstallomator:mainfrom
KarimAltavia:update-screamingfrog
Closed

Mise à jour du script pour Screaming Frog SEO avec screaming-frog-seo…#1900
KarimAltavia wants to merge 4 commits intoInstallomator:mainfrom
KarimAltavia:update-screamingfrog

Conversation

@KarimAltavia
Copy link
Contributor

…-spider.json
Mise à jour du script pour Screaming Frog SEO Spider

Cette modification améliore la gestion de Screaming Frog SEO Spider dans Installomator en utilisant l'API Homebrew pour obtenir dynamiquement la dernière version disponible. Les principaux changements sont :

  1. Utilisation de l'API Homebrew pour récupérer les informations de la dernière version.
  2. Détermination dynamique de l'architecture (x86_64 ou aarch64) pour télécharger la version appropriée.
  3. Construction dynamique de l'URL de téléchargement basée sur la version et l'architecture.

Ces modifications permettent :

  • Une mise à jour automatique de la version sans modification manuelle du script.
  • Une meilleure compatibilité avec différentes architectures de Mac.
  • Une réduction des risques d'erreurs liées à des versions obsolètes.

Le script a été testé sur macOS Ventura (x86_64 et Apple Silicon).

Update script for Screaming Frog SEO Spider

This modification enhances the handling of Screaming Frog SEO Spider in Installomator by using the Homebrew API to dynamically fetch the latest available version. The main changes are:

  1. Use of the Homebrew API to retrieve the latest version information.
  2. Dynamic determination of architecture (x86_64 or aarch64) to download the appropriate version.
  3. Dynamic construction of the download URL based on version and architecture.

These changes allow for:

  • Automatic version updates without manual script modification.
  • Better compatibility with different Mac architectures.
  • Reduced risk of errors related to outdated versions.

The script has been tested on macOS Ventura (x86_64 and Apple Silicon).

@KarimAltavia
Copy link
Contributor Author

Mise à jour du label Screaming Frog SEO Spider

Cette Pull Request met à jour le label pour Screaming Frog SEO Spider dans Installomator, améliorant la fiabilité et l'automatisation du processus d'installation.

Changements principaux

  1. Détection dynamique de la version : Le script extrait maintenant automatiquement la dernière version disponible depuis la page d'historique des versions de Screaming Frog.

  2. Support multi-architecture : Le script détecte automatiquement l'architecture de la machine (Apple Silicon ou Intel) et sélectionne l'URL de téléchargement appropriée.

  3. Extraction des URLs de téléchargement : Les URLs de téléchargement pour les versions Apple Silicon et Intel sont extraites directement de la page web, assurant que nous avons toujours les liens les plus récents.

  4. Vérification de l'URL de téléchargement : Une vérification a été ajoutée pour s'assurer qu'une URL valide a été trouvée, évitant ainsi les erreurs potentielles lors du téléchargement.

  5. Affichage des informations : Le script affiche maintenant des informations détaillées (nom, version, URL de téléchargement, ID d'équipe attendu) pour faciliter la vérification et le débogage.

Avantages

  • Mise à jour automatique : Plus besoin de mettre à jour manuellement le script à chaque nouvelle version de Screaming Frog SEO Spider.
  • Compatibilité améliorée : Fonctionne correctement sur les Mac Intel et Apple Silicon.
  • Fiabilité accrue : Réduit les risques d'erreurs liés à des URLs obsolètes ou incorrectes.

Tests effectués

Le script a été testé sur :

  • macOS Ventura (Intel)
  • macOS Ventura (Apple Silicon)

Il fonctionne correctement et télécharge la dernière version disponible de Screaming Frog SEO Spider pour chaque architecture.

@acodega
Copy link
Collaborator

acodega commented Sep 20, 2024

Thanks for the PR to improve the label. A few minor notes.

We don't like to have commenting in the app label, could you remove yours and also remove the blank lines?

We also don't like to have individual labels with their own log entries. Can you remove your echo commands, including checking whether $downloadURL is blank?

I think your variables like apple_silicon_url and etc aren't needed, so you can simplify it.. You can just put the code for that variable inside of the $downloadURL variable like this:

downloadURL="$(curl -s https://www.screamingfrog.co.uk/seo-spider/release-history/ | grep -o 'href="https://download.screamingfrog.co.uk/products/seo-spider/ScreamingFrogSEOSpider-[0-9.]*-aarch64.dmg"' | cut -d'"' -f2)"

You can do the same with appNewVersion.

appNewVersion="$(echo "$downloadURL" | grep -o '[0-9.]*-aarch64' | cut -d'-' -f1)"

I would put appNewVersion after downloadURL inside of the if statement, so it checks the version number for the respective platform. For example, the above appNewVersion would be inside of the if for Apple Silicon. Just in case the vendor ever releases a new version for Apple Silicon but not Intel. I've seen other vendors do that occasionally.

As an aside, if you are doing logging in labels in your own branch of Installomator, you should make use of the built-in printlog function.

Let me know what you think and if you want to update the PR. Thanks!

@acodega acodega added application adds or improves an application label waiting for response labels Sep 20, 2024
@acodega acodega added this to the v10.7 milestone Sep 20, 2024
Copy link
Contributor Author

@KarimAltavia KarimAltavia left a comment

Choose a reason for hiding this comment

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

Mise à jour du label Screaming Frog SEO Spider

Cette pull request améliore le label Screaming Frog SEO Spider dans Installomator en introduisant une détection dynamique de la version et une gestion améliorée des architectures.

Changements principaux

  1. Détection dynamique de la version :

    • Utilise la page d'historique des versions de Screaming Frog pour extraire automatiquement la dernière version disponible.
    • Élimine le besoin de mises à jour manuelles du script à chaque nouvelle version.
  2. Support multi-architecture :

    • Détecte automatiquement l'architecture de la machine (Apple Silicon ou Intel).
    • Construit dynamiquement l'URL de téléchargement appropriée basée sur l'architecture.
  3. Construction dynamique des URLs :

    • Génère les URLs de téléchargement basées sur la version extraite, assurant toujours le lien vers la dernière version.
  4. Vérifications de sécurité :

    • Inclut des vérifications pour s'assurer que la version et l'URL ont été correctement extraites.
    • Affiche des messages d'erreur clairs en cas de problème.
  5. Affichage des informations :

    • Affiche clairement le nom, la version, l'URL de téléchargement et l'ID d'équipe attendu pour faciliter le débogage.

Avantages

  • Maintenance réduite : Le script s'adapte automatiquement aux nouvelles versions sans intervention manuelle.
  • Fiabilité accrue : Utilise une source fiable (page d'historique des versions) pour obtenir les informations les plus récentes.
  • Flexibilité : S'adapte automatiquement aux changements d'URL ou de structure de version.
  • Compatibilité étendue : Fonctionne correctement sur les Mac Intel et Apple Silicon.

@scriptingosx scriptingosx modified the milestones: v10.7, v10.8 Jan 24, 2025
@scriptingosx
Copy link
Collaborator

Thank you!

@scriptingosx scriptingosx added the duplicate This issue or pull request already exists label Mar 21, 2025
@scriptingosx
Copy link
Collaborator

duplicate #1956

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

Labels

application adds or improves an application label duplicate This issue or pull request already exists

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants