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 gaianet-domain to gaia-frp #131

Closed
wants to merge 1 commit into from

Conversation

DarumaDocker
Copy link
Contributor

No description provided.

Copy link
Contributor

juntao commented Sep 4, 2024

Hello, I am a code review agent on flows.network. Here are my reviews of changed source code files in this PR.


gaianet

Potential issues

N/A

Summary of changes

  1. The script has been updated to modify and use the "frpc.toml" file located in a new directory called "gaia-frp" instead of "gaianet-domain".
  2. All references to "gaianet-domain" have been replaced with "gaia-frp" throughout the script, including log files, process IDs, and console output.
  3. The script now removes all files in the "gaia-frp" directory except for "frpc" and "frpc.toml", instead of doing so in the "gaianet-domain" directory.

install.sh

Potential issues

N/A

Summary of changes

  • The name of the main application has been changed from "gaianet-domain" to "gaia-frp".
  • The version number for gaia-frp is updated to v0.1.2.
  • All instances of "gaianet-domain" in the script have been replaced with "gaia-frp".
  • The frpc binary is now copied from $gaianet_base_dir/gaia-frp to $gaianet_base_dir/bin instead of $gaianet_base_dir/gaianet-domain.
  • All instances of "$gaianet_domain" in the script have been replaced with "$gaia_frp".

These changes are likely due to a renaming or rebranding of the main application, which includes updating the version number and adjusting file paths and references throughout the script accordingly. The frpc binary is also moved to a new location within the directory structure.

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.

3 participants