雑多なトーク #6
Replies: 33 comments 68 replies
-
|
Kickoffまでに何か準備することあります？ |
Beta Was this translation helpful? Give feedback.
-
|
それだ |
Beta Was this translation helpful? Give feedback.
-
|
機能洗い出しに使えそうなinputはあるといいなー。 |
Beta Was this translation helpful? Give feedback.
-
|
じゃあ宿題は、「UNIXシステムコール・プログラミングの6章」を読んで機能を考えておくにしましょー。 |
Beta Was this translation helpful? Give feedback.
-
|
子プロセスでpipeする件が腹落ちしてない。 |
Beta Was this translation helpful? Give feedback.
-
|
https://github.com/katataku/minishell/wiki/設計 |
Beta Was this translation helpful? Give feedback.
-
|
Kickoff何します？ |
Beta Was this translation helpful? Give feedback.
-
|
序盤の方に簡易的なプログラムと統合テストの骨格+Github Actionsは整備しておきたいかも。。 |
Beta Was this translation helpful? Give feedback.
-
|
それです！ |
Beta Was this translation helpful? Give feedback.
-
|
内容空っぽでも「とりあえずテストが動いている状態」までは持って行ってから本格的な非同期作業に入りたいですよね。 |
Beta Was this translation helpful? Give feedback.
-
|
賛成です。 |
Beta Was this translation helpful? Give feedback.
-
|
「プロンプトを表示して単純なコマンドを実行する」ところまでやるのがちょうどよいかも。 |
Beta Was this translation helpful? Give feedback.
-
|
冬のレビューキャンペーンってどうされます？参加するか迷ってるのですが、minishellの進行にも影響するなーと思っていて。 |
Beta Was this translation helpful? Give feedback.
-
|
僕はやらないつもりですー。 |
Beta Was this translation helpful? Give feedback.
-
|
ありがとうございます。片岡さんやらないならぼくもやらないでおこうかな〜。 |
Beta Was this translation helpful? Give feedback.
-
|
あ、bashのpdf化失敗してますね。 |
Beta Was this translation helpful? Give feedback.
-
このコマンドでやったのですが、デカすぎて何らかのエラーになってるとかですかね？ |
Beta Was this translation helpful? Give feedback.
-
|
できたかも。 |
Beta Was this translation helpful? Give feedback.
-
|
多段パイプだいたいできてきました〜。 |
Beta Was this translation helpful? Give feedback.
-
|
シグナル周りも何かしらしないといけなさそうな雰囲気 |
Beta Was this translation helpful? Give feedback.
-
|
コマンド実行中にシグナルを受け取った時の挙動。 |
Beta Was this translation helpful? Give feedback.
-
|
適当に言いました〜、たぶんそんな感じだったり、bashのプロンプト受付中とかのシグナル入力ですね〜。 |
Beta Was this translation helpful? Give feedback.
-
|
オシャレなウェルカムメッセージも作りたいかも |
Beta Was this translation helpful? Give feedback.
-
|
Beta Was this translation helpful? Give feedback.
-
|
草ww |
Beta Was this translation helpful? Give feedback.
-
|
変換ツール的なのあるんですか？ |
Beta Was this translation helpful? Give feedback.
-
|
https://tool-taro.com/image_to_ascii/ |
Beta Was this translation helpful? Give feedback.
-
|
マージしたら自動的にブランチが削除されるようにしたいかも。 |
Beta Was this translation helpful? Give feedback.
-
|
対応しましたー |
Beta Was this translation helpful? Give feedback.
-
|
マージしたと、issueって自分でクローズしないといけないんですか？ |
Beta Was this translation helpful? Give feedback.
-
|
そんなこともないです。 |
Beta Was this translation helpful? Give feedback.
-
|
あとデフォルトのマージをsquashマージにしたいかもです。ぼくSetting権限なく。 |
Beta Was this translation helpful? Give feedback.
-
|
すいません、間違えました☠️ squashマージだけ有効にしました！ |
Beta Was this translation helpful? Give feedback.
-
|
コメントのresolveはコメントした側がしたいかも。 |
Beta Was this translation helpful? Give feedback.
-
|
すいません！そうしましょう |
Beta Was this translation helpful? Give feedback.
-
|
42ヘッダーは今の時点からつけててもいいと思ったり。 |
Beta Was this translation helpful? Give feedback.
-
|
こだわりないので賛成です。 |
Beta Was this translation helpful? Give feedback.
-
|
pipexいい感じなんで良きタイミングで https://github.com/katataku/42_pipex_pair にPRあげますね |
Beta Was this translation helpful? Give feedback.
-
|
修正差分がありすぎてかなり見づらいかも。。 |
Beta Was this translation helpful? Give feedback.
-
|
読むのつらかったorz |
Beta Was this translation helpful? Give feedback.
-
|
OKです。肝心なところはここだけです〜。 |
Beta Was this translation helpful? Give feedback.
-
|
bashではなくてIEEEの規格を参照した方が良いかもという説。 |
Beta Was this translation helpful? Give feedback.
-
|
IEEEの規格がPOSIXということかな？ |
Beta Was this translation helpful? Give feedback.
-
のりさんのtimesより抜粋 |
Beta Was this translation helpful? Give feedback.
-
|
問題文が、 とはいえ、大方針として、「課題で要求されてるか微妙なラインのbash zshが勝手にやってる系エッジケース」は「Anything that is not asked is not required.」でディフェンスしましょ〜 |
Beta Was this translation helpful? Give feedback.
-
|
ヒアドキュメントも基本的なところはできた！ |
Beta Was this translation helpful? Give feedback.
-
|
実行部分の実装、だいたい終わりました〜。 |
Beta Was this translation helpful? Give feedback.
-
|
historyの記録はどこだろう？実はlexer？ |
Beta Was this translation helpful? Give feedback.
-
|
mainから別で呼ぶ感じかな？ |
Beta Was this translation helpful? Give feedback.
-
|
readlineにhistory機能あるので、それで賄えるかもって思ってます！ |
Beta Was this translation helpful? Give feedback.
-
|
学習という観点でいくと敢えて役割分担を途中で入れ替えるのもありかもしれない。 |
Beta Was this translation helpful? Give feedback.
-
|
面白いですね！ issueの途中で入れ替えると気持ち悪いので、 |
Beta Was this translation helpful? Give feedback.
-
|
やっちゃいますか。たぶん今までの感じを踏襲すると以下みたいになると思うんですけど次のチケット割り振りで逆にしちゃいますか。
|
Beta Was this translation helpful? Give feedback.
-
|
so_longでしげさんに書いてもらってた部分を自分で再実装しているのですが、意外とコード書かないと分からないこととかがあることに気づいたのがきっかけです。 |
Beta Was this translation helpful? Give feedback.
-
|
そうしましょう！ ヒアドキュメントあたりとか、そうなりそうな感じがしてたのでめっちゃ楽しみ！ |
Beta Was this translation helpful? Give feedback.
-
|
このエラーってどのタイミングで返すのがスマートですかね？ bash-3.2$ echo ${HO"ME"}
bash: ${HO"ME"}: bad substitution |
Beta Was this translation helpful? Give feedback.
-
|
変数名がおかしい場合ってことですよね。 |
Beta Was this translation helpful? Give feedback.
-
|
lexerよりかは変数の展開するやつの責務ですかね〜？ |
Beta Was this translation helpful? Give feedback.
-
|
bashってEOF入力されると終了するんですね。 |
Beta Was this translation helpful? Give feedback.
-
|
課題要件にもあった |
Beta Was this translation helpful? Give feedback.
-
|
今週のチケットは終わりました〜。分量かなり少なかったんで、Ctrl + CとCtrl + \の対応もやっちゃいます。 |
Beta Was this translation helpful? Give feedback.
-
|
イケメン！ |
Beta Was this translation helpful? Give feedback.
-
|
モジュールに属さないソースフアイルはどこに置きましょうかね？signal周りの処理はset_signal.cみたいなファイルに切り出せそうですが、どこに置こうかなと。srcs/直下ですかね。 |
Beta Was this translation helpful? Give feedback.
-
|
signalの例だけ聞くと、signalモジュール作っちゃってもいいかなと思いました。 それ以外でモジュール化難しければsrcs直下に賛成ですー |
Beta Was this translation helpful? Give feedback.
-
|
一旦雑にmain.cに入れてあります。singalの扱いやモジュールに切り出すのが難しいファイルの置き場所は来週話したいです〜 |
Beta Was this translation helpful? Give feedback.
-
|
次のタスク割り振りの時にプランニングポーカー的なのやってみてもいいかも。 |
Beta Was this translation helpful? Give feedback.
-
|
いいですねーやってみましょー！ |
Beta Was this translation helpful? Give feedback.
-
|
思いのほかシンプルにパイプ対応ができた！ 今週末に結合やってみますねー |
Beta Was this translation helpful? Give feedback.
-
|
indexの命名は好みが分かれてそう。 |
Beta Was this translation helpful? Give feedback.
-
|
合わせます！ |
Beta Was this translation helpful? Give feedback.
-
|
ありがとうございます。normで対応できない命名規則などは軽くdocsにまとめてもいいかも？ |
Beta Was this translation helpful? Give feedback.
-
|
気が向いたので、（優先度無視して）リダイレクト対応に手をつけてしまいましたー |
Beta Was this translation helpful? Give feedback.
-
|
気が向いて全部実装終わらせてもらっても良いですよ〜 |
Beta Was this translation helpful? Give feedback.
-
|
captureする代わりに出力先を変えることもできるかも？ |
Beta Was this translation helpful? Give feedback.
-
|
タブ補完もreadlineが勝手にやってくれてるのか。すごい。 |
Beta Was this translation helpful? Give feedback.
-
|
処理効率などを考えると環境変数はコピーオンライトが良さそう。ただ実装がちょっとめんどくさいので、対応はしないのが良さそう。 |
Beta Was this translation helpful? Give feedback.
-
|
./minishellプログラムのに引数が渡された場合の挙動どうしますか？ 対応せずに「引数があればエラー」でもいいかもと思ってます。 |
Beta Was this translation helpful? Give feedback.
-
|
引数は無視して、毎回対話モードで起動する感じでいいかなと思ってます。 |
Beta Was this translation helpful? Give feedback.
-
|
すいません〜間違えてmainにmergeしちゃったんでrevertしましたm(_ _)m |
Beta Was this translation helpful? Give feedback.
-
|
なんかこのdiscussion長くなっちゃったんで別で立てなおしてもいいかもですね。 |
Beta Was this translation helpful? Give feedback.
Uh oh!
There was an error while loading. Please reload this page.
-
どのトピックで話していいか迷ったらとりあえずここで。
Beta Was this translation helpful? Give feedback.
All reactions