Skip to content

Commit

Permalink
Remove deprecated configure option --with-xdgopen (#407)
Browse files Browse the repository at this point in the history
非推奨のconfigureオプション`--with-xdgopen`を削除します。
削除したオプションを指定しても無視されますので注意してください。
合わせてCIマトリックスの設定からxdg-openのオプションを取り除き
ソースコードを整理します。
  • Loading branch information
ma8ma committed Jul 19, 2020
1 parent 0f055a2 commit 535d167
Show file tree
Hide file tree
Showing 7 changed files with 6 additions and 42 deletions.
2 changes: 1 addition & 1 deletion .github/workflows/ccpp.yml
Original file line number Diff line number Diff line change
Expand Up @@ -142,7 +142,7 @@ jobs:
packages: libssl-dev libonig-dev libmigemo-dev
- config_args: --with-tls=gnutls --with-regex=pcre --with-alsa
packages: libgnutls28-dev libpcre3-dev libasound2-dev
- config_args: --with-tls=openssl --with-regex=glib --with-xdgopen
- config_args: --with-tls=openssl --with-regex=glib
packages: libssl-dev
steps:
- uses: actions/checkout@v2
Expand Down
4 changes: 2 additions & 2 deletions .travis.yml
Original file line number Diff line number Diff line change
Expand Up @@ -10,10 +10,10 @@ cache:

matrix:
include:
- name: "CONFIG_ARG=\"--with-tls=gnutls --with-regex=posix --with-migemo --with-xdgopen\""
- name: "CONFIG_ARG=\"--with-tls=gnutls --with-regex=posix --with-migemo\""
os: linux
compiler: gcc
env: CONFIG_ARG="--with-tls=gnutls --with-regex=posix --with-migemo --with-xdgopen"
env: CONFIG_ARG="--with-tls=gnutls --with-regex=posix --with-migemo"
addons:
apt:
update: true
Expand Down
5 changes: 0 additions & 5 deletions INSTALL
Original file line number Diff line number Diff line change
Expand Up @@ -96,11 +96,6 @@

ALSAによる効果音再生機能を有効にする。詳しくは"https://jdimproved.github.io/JDim/"の項を参照すること。

--with-xdgopen

デフォルトブラウザとしてxdg-openを使用する。
xdg-openは将来のリリースでデフォルトブラウザになりこのオプションは廃止される予定です。

--enable-gprof

gprofによるプロファイリングを行う。コンパイルオプションに -pg が付き、JDimを実行すると
Expand Down
16 changes: 0 additions & 16 deletions configure.ac
Original file line number Diff line number Diff line change
Expand Up @@ -318,22 +318,6 @@ AS_IF(
)


dnl
dnl checking xdg-open
dnl
AC_MSG_CHECKING(for --with-xdgopen)
AC_ARG_WITH(xdgopen,
AC_HELP_STRING([--with-xdgopen],[use xdg-open as default browser]),
[with_xdgopen="$withval"], [with_xdgopen=no])
AC_MSG_RESULT($with_xdgopen)

AS_IF(
[test "x$with_xdgopen" = xyes],
[AC_DEFINE(XDGOPEN, , [use xdg-open])
AC_MSG_WARN([--with-xdgopen will be removed and as default in the future release.])]
)


dnl
dnl checking alsa
dnl
Expand Down
5 changes: 0 additions & 5 deletions docs/manual/make.md
Original file line number Diff line number Diff line change
Expand Up @@ -106,11 +106,6 @@ configure のかわりに [meson] を使ってビルドする方法は [GitHub][
<dt>--with-alsa</dt>
<dd>ALSA による効果音再生機能を有効にする。
詳しくは<a href="{{ site.baseurl }}/sound/">効果音の再生</a>の項を参照すること。</dd>
<dt>--with-xdgopen</dt>
<dd>
デフォルトブラウザとしてxdg-openを使用する。<br>
xdg-openは将来のリリースでデフォルトブラウザになりこのオプションは廃止される予定です。
</dd>
<dt>--enable-gprof</dt>
<dd>
gprof によるプロファイリングを行う。
Expand Down
3 changes: 0 additions & 3 deletions meson.build
Original file line number Diff line number Diff line change
Expand Up @@ -216,9 +216,6 @@ else
message('Use compatible cache directory: NO')
endif

# xdg-open
conf.set('XDGOPEN', 1)


#
# コンパイラーの追加オプション
Expand Down
13 changes: 3 additions & 10 deletions src/config/defaultconf.h
Original file line number Diff line number Diff line change
Expand Up @@ -167,18 +167,11 @@ namespace CONFIG
};

// browsers.cpp のデフォルトのラベル番号
// configure で --with-xdg-open を指定すると xdg-open をデフォルトにする
// browsers.cpp のブラウザの順番に気をつけること
enum{
#ifndef _WIN32
#ifdef XDGOPEN
CONF_BROWSER_NO = 1 // xdg-open をデフォルトにする
#else
CONF_BROWSER_NO = 2 // firefox をデフォルトにする
#endif
#else /* _WIN32 */
CONF_BROWSER_NO = 1 // ie をデフォルトにする
#endif /* _WIN32 */
// _WIN32 : ie をデフォルトにする
// not _WIN32 : xdg-open をデフォルトにする
CONF_BROWSER_NO = 1
};

#define CONF_FONTSIZE_THREAD "12"
Expand Down

0 comments on commit 535d167

Please sign in to comment.