surfstudio / ReactiveDataDisplayManager

MIT License
34 stars 13 forks source link

Update gems on arm64 #222

Closed Ikeret closed 1 year ago

Ikeret commented 1 year ago

Что сделано?

Зачем это сделано?

На что обратить внимание?

github-actions[bot] commented 1 year ago
Messages
:book: Great! We didn't found any violations in your changes. Congratulations 🎉

Generated by :no_entry_sign: Danger Swift against 96409f144994234c37a96b811742ace18b0ebfd3

codecov-commenter commented 1 year ago

Codecov Report

Patch and project coverage have no change.

Comparison is base (7577eca) 69.68% compared to head (96409f1) 69.68%.

Additional details and impacted files ```diff @@ Coverage Diff @@ ## develop #222 +/- ## ======================================== Coverage 69.68% 69.68% ======================================== Files 141 141 Lines 4153 4153 Branches 1876 1876 ======================================== Hits 2894 2894 Misses 1161 1161 Partials 98 98 ``` | Flag | Coverage Δ | | |---|---|---| | uitests | `57.18% <ø> (ø)` | | | unittests | `39.73% <ø> (ø)` | | Flags with carried forward coverage won't be shown. [Click here](https://docs.codecov.io/docs/carryforward-flags?utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=surfstudio#carryforward-flags-in-the-pull-request-comment) to find out more.

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Do you have feedback about the report comment? Let us know in this issue.

Ikeret commented 1 year ago

Выяснил, что на самом деле из хука команда вызывается, но не работает правильно

$ git checkout release/7.4

Switched to branch 'release/7.4'
xcodegen generate
⚙️  Generating plists...
⚙️  Generating project...
⚙️  Writing project...
Created project at /Users/korshunov/Documents/ReactiveDataDisplayManager/ReactiveDataDisplayManager.xcodeproj
bundle exec pod install
/System/Library/Frameworks/Ruby.framework/Versions/2.6/usr/lib/ruby/2.6.0/rubygems.rb:283:in `find_spec_for_exe': Could not find 'bundler' (2.4.6) required by your /Users/korshunov/Documents/ReactiveDataDisplayManager/Gemfile.lock. (Gem::GemNotFoundException)
To update to the latest version installed on your system, run `bundle update --bundler`.
To install the missing version, run `gem install bundler:2.4.6`
    from /System/Library/Frameworks/Ruby.framework/Versions/2.6/usr/lib/ruby/2.6.0/rubygems.rb:302:in `activate_bin_path'
    from /usr/local/bin/bundle:23:in `<main>'
Your branch is up to date with 'origin/release/7.4'.
make: [projects] Error 1 (ignored)

Логи можно посмотреть с помощью Activity Manager в Fork

Снимок экрана 2023-05-29 в 15 44 38
Ikeret commented 1 year ago

На Intel маке make projects не производит дифф c этими изменениями. Добавил bindle install в make projects чтобы гемы ставились, но хуки все равно может не починятся, потому что fork тянет ruby 2.6.0 с какого-то перепугу

он тянет системный, а не из homebrew, игнорируя профайлы. Как это починить непонятно)

image
Ikeret commented 1 year ago

В $PATH гит шела нет ничего из указанного в .bash_profile. Тоже самое для системного инстанса гита

/Applications/Fork.app/Contents/Resources/git-instance/libexec/git-core:/Applications/Fork.app/Contents/Resources/git-instance/git-lfs:/Applications/Fork.app/Contents/Resources/gitflow-avh:/opt/homebrew/bin:/usr/local/bin:/usr/bin:/bin:/usr/sbin:/sbin

Я искал способ как можно добавить туда путь до руби, но все пишут, что для non-interactive shell используется .bashrc. Оттуда в путь тоже ничего не попадает, поэтому добавил хак с реэкспортом путей из .bash_profile и все стало работать

image

Если решение ок, то надо и в 7.4 тоже обновить хуки

NullIsOne commented 1 year ago

В $PATH гит шела нет ничего из указанного в .bash_profile. Тоже самое для системного инстанса гита

/Applications/Fork.app/Contents/Resources/git-instance/libexec/git-core:/Applications/Fork.app/Contents/Resources/git-instance/git-lfs:/Applications/Fork.app/Contents/Resources/gitflow-avh:/opt/homebrew/bin:/usr/local/bin:/usr/bin:/bin:/usr/sbin:/sbin

Я искал способ как можно добавить туда путь до руби, но все пишут, что для non-interactive shell используется .bashrc. Оттуда в путь тоже ничего не попадает, поэтому добавил хак с реэкспортом путей из .bash_profile и все стало работать image

Если решение ок, то надо и в 7.4 тоже обновить хуки

Слил и подтянул вветку с прототипом чата - хуки сработали так как у меня тоже bash_profile источник для переменных.