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

fix: remove env SQLE_JAVA_HOME #401

Merged
merged 1 commit into from
Mar 27, 2025
Merged

fix: remove env SQLE_JAVA_HOME #401

merged 1 commit into from
Mar 27, 2025

Conversation

littleniannian
Copy link
Collaborator

@littleniannian littleniannian commented Mar 26, 2025

User description

issue: actiontech/sqle#2995
link actiontech/sqle#2995


Description

  • 删除 .bashrc 中对 SQLE_JAVA_HOME 的配置

  • 直接在 systemd 服务文件中设置 JAVA_HOME

  • 优化 systemd 服务配置脚本


Changes walkthrough 📝

Relevant files
Enhancement
dms_sqle_provision.spec
优化 `build/dms_sqle_provision.spec` 中的 JAVA_HOME 设置             

build/dms_sqle_provision.spec

  • 删除 .bashrc 中关于 SQLE_JAVA_HOME 的设置
  • 在 systemd 服务文件中直接设置 JAVA_HOME$RPM_INSTALL_PREFIX/jdk
  • 更新 systemd 服务配置以反映更改
  • +1/-14   

    Need help?
  • Type /help how to ... in the comments thread for any questions about PR-Agent usage.
  • Check out the documentation for more information.
  • @actiontech-bot actiontech-bot requested review from iwanghc and removed request for iwanghc March 26, 2025 07:04
    Copy link

    PR Reviewer Guide 🔍

    ⏱️ Estimated effort to review: 2 🔵🔵⚪⚪⚪
    🧪 No relevant tests
    🔒 No security concerns identified
    ⚡ No major issues detected

    Copy link

    PR Code Suggestions ✨

    No code suggestions found for the PR.

    @iwanghc iwanghc merged commit 50cae8f into main Mar 27, 2025
    1 check passed
    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.

    2 participants