eclass: add inline documentation, fix some pkgcheck warnings #38

Merged
lautre merged 1 commits from :eclasses into develop 1 year ago
ktrace commented 1 year ago
  • исправлены пробелы/табуляция
  • добавлены описания функций и некоторых переменных
  • исправлено предупреждение об неэкранированной переменной
- исправлены пробелы/табуляция - добавлены описания функций и некоторых переменных - исправлено предупреждение об неэкранированной переменной
ktrace force-pushed eclasses from cf600d07d9 to 869080475b 1 year ago
lautre commented 1 year ago
Owner

Здесь добавление кавычек приведет к другому результату. Потом снова будем удивляться почему nvidia-drivers не работают:
cp -a "${ROOT}/usr/share/${PN}/${PV}/firmware/*" "${ROOT}/lib/firmware/"

Здесь добавление кавычек приведет к другому результату. Потом снова будем удивляться почему nvidia-drivers не работают: `cp -a "${ROOT}/usr/share/${PN}/${PV}/firmware/*" "${ROOT}/lib/firmware/"`
ktrace force-pushed eclasses from 869080475b to e1d077c8ab 1 year ago
ktrace commented 1 year ago
Poster

Здесь добавление кавычек приведет к другому результату.

поправил

> Здесь добавление кавычек приведет к другому результату. поправил
kuraga commented 1 year ago

Берите просто каждую переменную в кавычки.

cp -a "${ROOT}"/usr/share/"${PN}"/"${PV}"/firmware/* "${ROOT}"/lib/firmware/
Берите просто каждую переменную в кавычки. ``` cp -a "${ROOT}"/usr/share/"${PN}"/"${PV}"/firmware/* "${ROOT}"/lib/firmware/ ```
ktrace commented 1 year ago
Poster

по большому счёту критична только *.

по большому счёту критична только *.
kuraga commented 1 year ago

@ktrace , ага. Только думай каждый раз, что критично. Explicit is better than implicit.

Хотя поправка. Более "просто" брать целиком. Ладно, я сам запутался...

@ktrace , ага. Только думай каждый раз, что критично. Explicit is better than implicit. Хотя поправка. Более "просто" брать целиком. Ладно, я сам запутался...
kuraga commented 1 year ago

@ktrace , да! Я именно совпадаю с тобой по мнению! Сам запутался...

P.S. Когда-то запомнил, что "феномен" надо говорить не так, как хочется... Ох и ошибкой это было... Так же и с кавычками.

@ktrace , да! Я именно совпадаю с тобой по мнению! Сам запутался... P.S. Когда-то запомнил, что "феномен" надо говорить не так, как хочется... Ох и ошибкой это было... Так же и с кавычками.
lautre merged commit 606d537387 into develop 1 year ago
The pull request has been merged as 606d537387.
Sign in to join this conversation.
No reviewers
No Label
No Milestone
No Assignees
3 Participants
Notifications
Due Date

No due date set.

Dependencies

No dependencies set.

Reference: calculate/calculate-overlay#38
Loading…
There is no content yet.