search

rust-lang-ru / cargo-docs-ru

Перевод документации по Cargo на русский язык
Apache License 2.0
12 stars 3 forks source link

Перевод главы Build Script support #6

Closed chabapok closed 7 years ago

chabapok commented 7 years ago

This change is Reviewable

mkpankov commented 7 years ago

@chabapok спасибо за PR!

@Gordon-F сможешь просмотреть?

Gordon-F commented 7 years ago

@chabapok Спасибо! Review постараюсь сделать на выходных.

chabapok commented 7 years ago

Да. Перевод далеко не идеален. Сначала переводил для себя, цель была разобраться, а не соблюсти точность. Так что, весьма вероятно, над переводом понадобится еще работать. Я не знаю какая тут у нас принята стратегия принятия подобных пулл реквестов. То ли наполовину сделанное принимается, то ли нет. Поэтому спасибо говорить еще рано. Может быть и так, что там окажется ну совсем все плохо.

Кроме того, вышедшая вчера версия rust (вроде бы) добавляет фичу автоопределения build.rs, а если ты хочешь чтобы он не подхватывался, то надо писать build=false. В переводе про это ничего нет. Надо добавлять.

Gordon-F commented 7 years ago

@chabapok Думаю, что лучше тогда добавить WIP к названию PR. После того, как обновишь перевод можно будет сделать полноценное review.

chabapok commented 7 years ago

Что такое WIP, что такое PR?

Gordon-F commented 7 years ago

@chabapok WIP - Work in progress. PR - pull request.

chabapok commented 7 years ago

WIP добавил. Но вряд ли я буду обновлять перевод, по крайней мере, в ближайшее время по одной простой причине: чтобы перевести лучше, нужно уметь это делать. Так что, если в таком виде как есть сейчас, перевод не подходит для дальнейшей доработки - отменяйте.

chabapok commented 7 years ago

Ок, на днях сделаю и отпишусь сюда.

chabapok commented 7 years ago

crate -> контейнер artifact -> артефакт package -> пакет manifest -> манифест (предполагается файл Cargo.toml) build script -> сборочный скрипт, файл build.rs

Я не очень понимаю, в чем отличие артефакта от пакета и от контейнера. Помоему, это все синонимы,а слово artifact вообще из java пришло.

Помоему, такое кол-во синонимов в технической документации только запутывает.

Насчет этого: library -> либа, библиотека. Слово "либа", по моему мнению - хоть и жаргонное, но прижившееся настолько, что зачастую более понятно чем "библиотека".

На данный момент больше работ не планируется, так что можно ревьювить дальше.

chabapok commented 7 years ago

@Gordon-F @mkpankov ауау! Пулл реквест больше месяца висит уже. Это просто времени нет, или про него забыли? Я поправил все замечания (см. первомайский камент)

mkpankov commented 7 years ago

@chabapok прости за задержку, принимаем

Reviewed 1 of 2 files at r1, 1 of 1 files at r2. Review status: all files reviewed at latest revision, all discussions resolved.

Comments from Reviewable

chabapok commented 6 years ago

Сорри за некропост, у меня глупый вопрос.

А заголовок теперь так и будет висеть с префиксом [WIP]? У всех нету этого wip, я его должен убрать?

Gordon-F commented 6 years ago

@chabapok PR уже слит в мастер. Это не так важно.

chabapok commented 6 years ago

@Gordon-F он в мастер слит с этим префиксом, и после него там уже закоммитили еще.

Ладно, я попробую. Мне интересно, что из этого получится.

Так, и еще. Я обнаружил, что этот репозиторий форкнут из оригинального cargo. С него были удалены все исходники, оставили только документацию - и погнали документацию переводить. А основная репа развивалась, с тех пор в основной репе добавилось около 2тыс коммитов.

В свете этого, вопрос. У нас есть какой-то план начет того, как мы будем следить за новыми фичами в документации из основной репы?

Gordon-F commented 6 years ago

@chabapok Документация cargo переехала на mdbook. И теперь она переехала в репозиторий rust, насколько мне известно