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

CLDR-14316 doc: document SurveyTool setup #3135

Merged
merged 1 commit into from
Dec 5, 2023

Conversation

srl295
Copy link
Member

@srl295 srl295 commented Aug 1, 2023

CLDR-14213

  • This PR completes the ticket.

ALLOW_MANY_COMMITS=true

@srl295 srl295 requested a review from btangmu August 1, 2023 17:21
@srl295 srl295 self-assigned this Aug 1, 2023
@srl295
Copy link
Member Author

srl295 commented Aug 1, 2023

wip…

@btangmu
Copy link
Member

btangmu commented Aug 11, 2023

see also: https://docs.google.com/document/d/1wkJsoQXPcGN_dlCzXzwhfjQoq9Mes7VezIRmNrHWV8c/edit

I just added the following sentence to that doc, which otherwise hasn't been edited for a long time: 'Note: Tom has switched from Eclipse to IntelliJ in recent months, and debugs ST by starting it in Terminal, then choosing “Attach to Process” from the “Run” menu in IntelliJ.'

@srl295
Copy link
Member Author

srl295 commented Aug 24, 2023

@btangmu I'm planning to leave this open until after the merge (no reason to put it into 44)

@srl295 srl295 marked this pull request as ready for review September 19, 2023 22:16
@srl295 srl295 changed the title CLDR-14213 doc: document SurveyTool setup v45 CLDR-14213 doc: document SurveyTool setup Sep 19, 2023
@srl295
Copy link
Member Author

srl295 commented Sep 19, 2023

@btangmu I'd like to merge this in v45 so moving out of draft

@srl295 srl295 changed the title v45 CLDR-14213 doc: document SurveyTool setup CLDR-14213 doc: document SurveyTool setup Nov 29, 2023
@srl295 srl295 changed the title CLDR-14213 doc: document SurveyTool setup CLDR-14316 doc: document SurveyTool setup Nov 29, 2023
@jira-pull-request-webhook
Copy link

Hooray! The files in the branch are the same across the force-push. 😃

~ Your Friendly Jira-GitHub PR Checker Bot

@jira-pull-request-webhook
Copy link

Notice: the branch changed across the force-push!

  • tools/cldr-apps/README.md is different

View Diff Across Force-Push

~ Your Friendly Jira-GitHub PR Checker Bot

@srl295
Copy link
Member Author

srl295 commented Nov 29, 2023

@btangmu PTAL. If this is an improvement let's merge it otherwise let me know what you'd like to see. You had asked about logging for example.

Copy link
Member

@btangmu btangmu left a comment

Choose a reason for hiding this comment

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

LGTM -- FYI my own local setup has "root" instead of "surveytool" for the mysql user -- the username doesn't matter as long as you're consistent

also, I always import an existing db from a dump/backup rather than creating a new db, and that might be worthwhile to suggest as an alternative in this doc

My server.env also has these two lines:

WLP_DEBUG_SUSPEND=n
WLP_DEBUG_ADDRESS=7777

-- I'm not sure if they're needed, and I don't remember when/why I added them

A link to this doc might also be helpful here, even though it's also outdated: https://docs.google.com/document/d/1wkJsoQXPcGN_dlCzXzwhfjQoq9Mes7VezIRmNrHWV8c/edit#heading=h.ozmz4163qnxu

@srl295
Copy link
Member Author

srl295 commented Nov 29, 2023

you can run it as root, but i wouldn't recommend it. particularly, we might miss if a SQL action requires elevated privileges. some of the forum report code manipulates stored procedures for example.

is that document owned by the U+CLDR-TC drive? If not would you consider moving it there?

I don't know what the suspend is, but https://www.ibm.com/docs/en/was-liberty/core?topic=liberty-customizing-environment says that 7777 is the default port.

@srl295 srl295 merged commit 37cfe90 into unicode-org:main Dec 5, 2023
4 checks passed
@srl295 srl295 deleted the st-setup-docs branch December 5, 2023 19:04
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