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: Check dependencies in the prefix path before the default path #317

Merged
merged 1 commit into from
Nov 20, 2024

Conversation

namsic
Copy link
Collaborator

@namsic namsic commented Nov 20, 2024

๐Ÿ”— Related Issue

โŒจ๏ธ What I did

  • system path๋ณด๋‹ค prefix ๊ฒฝ๋กœ์— ์„ค์น˜๋œ ์˜์กด์„ฑ์„ ์šฐ์„  ํƒ์ƒ‰ํ•˜๋„๋ก ํ•ฉ๋‹ˆ๋‹ค.

@jhpark816
Copy link
Contributor

jhpark816 commented Nov 20, 2024

@namsic
ํ˜„์žฌ library ์ฐพ๋Š” ๊ณผ์ •์—์„œ ์–ด๋–ค ๋ฌธ์ œ๊ฐ€ ๋ฐœ์ƒํ•˜์˜€๋‚˜์š”?
์•„๋‹ˆ๋ฉด ๋…ผ๋ฆฌ์ ์œผ๋กœ prefix ๋จผ์ € ์ฐพ๋„๋ก ์ˆ˜์ •ํ•˜๋Š” ๊ฒƒ์ด ๋‚ซ๋‹ค๊ณ  ํŒ๋‹จํ•˜์—ฌ ์ˆ˜์ •ํ•˜๋Š” ๊ฒƒ์ธ๊ฐ€์š”?

@jhpark816
Copy link
Contributor

@namsic
์ผ๋‹จ mergeํ•ฉ๋‹ˆ๋‹ค.
C client์—์„œ library ์ฐพ๋Š” ๊ณผ์ •์—์„œ ์–ด๋–ค ์ด์Šˆ๋ฅผ ๊ฒฝํ—˜ํ–ˆ๋‹ค๋ฉด, ๋ณ„๋„๋กœ ๊ณต์œ ํ•ด ์ฃผ์„ธ์š”.

@jhpark816 jhpark816 merged commit 47995cd into naver:develop Nov 20, 2024
1 check passed
@namsic
Copy link
Collaborator Author

namsic commented Nov 20, 2024

ํ˜„์žฌ library ์ฐพ๋Š” ๊ณผ์ •์—์„œ ์–ด๋–ค ๋ฌธ์ œ๊ฐ€ ๋ฐœ์ƒํ•˜์˜€๋‚˜์š”?
์•„๋‹ˆ๋ฉด ๋…ผ๋ฆฌ์ ์œผ๋กœ prefix ๋จผ์ € ์ฐพ๋„๋ก ์ˆ˜์ •ํ•˜๋Š” ๊ฒƒ์ด ๋‚ซ๋‹ค๊ณ  ํŒ๋‹จํ•˜์—ฌ ์ˆ˜์ •ํ•˜๋Š” ๊ฒƒ์ธ๊ฐ€์š”?

ํŠน๋ณ„ํžˆ ๋ฌธ์ œ๋ฅผ ๊ฒช์—ˆ๋‹ค๊ธฐ๋ณด๋‹ค๋Š”, ์•„๋ž˜์™€ ๊ฐ™์€ ์ด์œ ์ž…๋‹ˆ๋‹ค.

  • ์บ์‹œ ์„œ๋ฒ„์™€ ๋นŒ๋“œ ๋ฐฉ์‹์— ์ผ๊ด€์„ฑ์„ ๊ฐ€์ ธ๊ฐ€๊ธฐ ์œ„ํ•จ์ž…๋‹ˆ๋‹ค.

    • ์บ์‹œ ์„œ๋ฒ„์˜ ๋นŒ๋“œ ๋ฐฉ์‹(prefix ์šฐ์„  ํƒ์ƒ‰)์„ ์•Œ๊ณ  ์žˆ๋Š” ์‚ฌ์šฉ์ž ์ž…์žฅ์—์„œ ํด๋ผ์ด์–ธํŠธ๋„ ๊ฐ™์€ ๋ฐฉ์‹์œผ๋กœ ๋นŒ๋“œํ•  ๊ฒƒ์ด๋ผ๊ณ  ๊ธฐ๋Œ€ํ•  ์ˆ˜ ์žˆ์Šต๋‹ˆ๋‹ค.
    • ์บ์‹œ ์„œ๋ฒ„์˜ ๋นŒ๋“œ ๋ฐฉ์‹์„ ๋ชจ๋ฅด๋Š” ์‚ฌ์šฉ์ž๋ผ๊ณ  ํ•˜๋”๋ผ๋„, prefix์— ๋ผ์ด๋ธŒ๋Ÿฌ๋ฆฌ๊ฐ€ ์„ค์น˜๋˜์–ด ์žˆ๋‹ค๋ฉด ํ•ด๋‹น ๋ผ์ด๋ธŒ๋Ÿฌ๋ฆฌ๋ฅผ ์‚ฌ์šฉํ•  ๊ฒƒ์ด๋ผ๊ณ  ์œ ์ถ”ํ•  ๊ฒƒ์ž…๋‹ˆ๋‹ค.
      jam2in/arcus-works#448
  • ํ˜„์žฌ๋Š” ์™ธ๋ถ€ ์˜์กด์„ฑ์„ libzookeeper ํ•˜๋‚˜๋งŒ ์‚ฌ์šฉํ•˜๊ณ  ์žˆ์ง€๋งŒ, ๋‹ค๋ฅธ ์˜์กด์„ฑ(sasl, ssl)์ด ์ถ”๊ฐ€๋œ๋‹ค๋ฉด ์œ„์™€ ๋™์ผํ•œ ํ˜•ํƒœ์˜ ์ด์Šˆ(segfault)๊ฐ€ ๋ฐœ์ƒํ•  ์ˆ˜ ์žˆ์„ ๊ฒƒ์ž…๋‹ˆ๋‹ค.

@namsic namsic deleted the prior branch November 20, 2024 12:16
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