Rietveld Code Review Tool
Help | Bug tracker | Discussion group | Source code | Sign in
(625)

Issue 9413043: Add man files of /usr/bin files. (Closed)

Can't Edit
Can't Publish+Mail
Start Review
Created:
11 years ago by fujiwara
Modified:
11 years ago
Reviewers:
shawn.p.huang, Peng
CC:
shawn.p.huang_gmail.com, fujiwara
Base URL:
git@github.com:ibus/ibus.git@master
Visibility:
Public.

Description

Add man files of /usr/bin files.

Patch Set 1 #

Total comments: 7

Patch Set 2 : Updated with message #2. #

Unified diffs Side-by-side diffs Delta from patch set Stats (+222 lines, -6 lines) Patch
M bus/Makefile.am View 1 4 chunks +19 lines, -2 lines 0 comments Download
A bus/ibus-daemon.1.in View 1 1 chunk +77 lines, -0 lines 0 comments Download
M setup/Makefile.am View 1 2 chunks +16 lines, -2 lines 0 comments Download
A setup/ibus-setup.1.in View 1 1 chunk +33 lines, -0 lines 0 comments Download
M tools/Makefile.am View 1 2 chunks +19 lines, -2 lines 0 comments Download
A tools/ibus.1.in View 1 1 chunk +58 lines, -0 lines 0 comments Download

Messages

Total messages: 3
fujiwara
11 years ago (2013-05-14 14:18:23 UTC) #1
Peng
lgtm with several comments https://codereview.appspot.com/9413043/diff/1/bus/Makefile.am File bus/Makefile.am (right): https://codereview.appspot.com/9413043/diff/1/bus/Makefile.am#newcode6 bus/Makefile.am:6: # Copyright (c) 2007-2010 Red ...
11 years ago (2013-05-14 14:42:22 UTC) #2
fujiwara
11 years ago (2013-05-15 02:50:48 UTC) #3
https://codereview.appspot.com/9413043/diff/1/bus/Makefile.am
File bus/Makefile.am (right):

https://codereview.appspot.com/9413043/diff/1/bus/Makefile.am#newcode6
bus/Makefile.am:6: # Copyright (c) 2007-2010 Red Hat, Inc.
On 2013/05/14 14:42:22, Peng wrote:
> Update the date in license header

Done.

https://codereview.appspot.com/9413043/diff/1/bus/ibus-daemon.1
File bus/ibus-daemon.1 (right):

https://codereview.appspot.com/9413043/diff/1/bus/ibus-daemon.1#newcode6
bus/ibus-daemon.1:6: .TH IBUS-DAEMON "1" "April 2010" "ibus-daemon 1.3.1" "User
Commands"
On 2013/05/14 14:42:22, Peng wrote:
> Is the version right?

OK, I renamed *.1 to *.1.in to save the version numbers.
Sign in to reply to this message.

Powered by Google App Engine
RSS Feeds Recent Issues | This issue
This is Rietveld f62528b