Bug 241822

Summary: [PATCH]Update port: japanese/font-ipamjm
Product: Ports & Packages Reporter: Tomoaki AOKI <junchoon>
Component: Individual Port(s)Assignee: Hiroki Sato <hrs>
Status: New ---    
Severity: Affects Only Me CC: tatsuki_makino
Priority: --- Keywords: buildisok
Version: LatestFlags: bugzilla: maintainer-feedback? (hrs)
Hardware: Any   
OS: Any   
Attachments:
Description Flags
Update to 006.01
none
patch for japanese/font-ipamjm tatsuki_makino: maintainer-approval?

Description Tomoaki AOKI 2019-11-09 13:48:29 UTC
Created attachment 208997 [details]
Update to 006.01

japanese/font-ipamjm was marked as BROKEN due to unfetchable.
This is already updated to 006.01 upstream and is fetchable.
Sync to upstream not to be deleted.
Comment 1 Automation User 2019-11-24 04:48:19 UTC
Build info is available at https://gitlab.com/swills/freebsd-ports/pipelines/98100604
Comment 2 Tatsuki Makino 2019-12-03 01:15:43 UTC
Created attachment 209636 [details]
patch for japanese/font-ipamjm

佐藤氏が適切に処理してくださると思うのですが、他のコミッターさんでもコミットできるパッチを用意するにあたってこちらの方が良いのではないかと思う点がいくつかあったので、代案として私からもパッチを上げさせていただきます。
・内部にディレクトリを持たず、ファイルをぶちまけてくださるdistfileには、NO_WRKSUBDIRを使うらしい。
・SUB_FILESに特定の名前のファイルを指定すると、自動的にWRKDIRに生成されたものを使うように変数を書き換える(Mk/bsd.port.mkのapply-slistターゲットのあたり)らしいので、そのあたりを省略。
・Readme.txtはありきたりな名前だが、 /usr/local/share/doc/ipamjm というパッケージ別のディレクトリへインストールするので、特に名前の重複を気にする必要はなさそう。
・distinfoはmake makesumを実行して生成するのが今の主流らしい。 (ただ、コミットまでに時間が経つとアレなので、TIMESTAMPをちょっと未来のキリがいいところに書き換えちゃってますけどw)

Summary of sentences written in Japanese.
NO_WRKSUBDIR should be used for distfile which doesn't have name-version directory.
PKGINSTALL=${WRKDIR}/pkg-install is automatically assigned, so it is omitted.
Use the name of Readme.txt without changing it.
Use the distinfo file generated by make makesum.