解决 macOS 10.12 下 PAC 模式下Safari 或 Twitter 客户端 tweetbot客户端不能FQ的问题

[qinyuhang]

解决 macOS 10.12 下 PAC 模式下Safari 或 Twitter 客户端 tweetbot客户端不能FQ的问题
解决 #42 #30 macOS 10.12 下 PAC 模式下Safari 或 Twitter 客户端 tweetbot客户端不能FQ的问题

PS 手头暂时没有 macOS 10.12 的机器，麻烦有的请帮忙测试一下
或者可以从 https://github.com/qinyuhang/ShadowsocksX-NG/tree/hotfix-macOS10.12 下载去
Xcode build 一下

[qiuyuzhou]

@qinyuhang 这次确认没问题？

对于是否将Pods放入版本控制系统的问题，2种方法各有优劣。将对podfile和pods下的修改单独进行提交，不要和主体的代码放在同一个commit里，就好多了。

[qinyuhang]

确定没问题了。
以后再写我会把pod和主题commit拆开来。谢谢。

[qiuyuzhou]

能否在高级配置里加一项，指定listen的端口。

[qinyuhang]

制定gcdwebserver的端口吗？我明天就加上。

Qiu Yuzhou notifications@github.com于2016年8月24日 周三23:01写道：

能否在高级配置里加一项，指定listen的端口。

—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
#44 (comment),
or mute the thread
https://github.com/notifications/unsubscribe-auth/AF8zUTL5bCZn_M9_e20aYpLARJo_i-jAks5qjFy-gaJpZM4JquMZ
.

[qinyuhang]

我看见在高级设置里面，所有的textField是直接和defaults bind到一起的，现在假设在GUI里面改变了任何一个textField，会触发哪个钩子函数么？这里加上了本地监听port之后需要重启gcdwebserver，我想在钩子函数里面加上，但是没有发现。

[qiuyuzhou]

现在是在窗口关闭的时候，发一个通知，不是很好，但是工作。

    //------------------------------------------------------------
    // NSWindowDelegate
    func windowWillClose(notification: NSNotification) {
        NSNotificationCenter.defaultCenter()
            .postNotificationName(NOTIFY_ADV_CONF_CHANGED, object: nil)
    }

[qinyuhang]

原来这个viewcontroller里面的函数是做这个用的呀。那我先用这个。同时也想想怎么做个钩子函数，以后把这个钩子函数push上去。

Qiu Yuzhou notifications@github.com于2016年8月25日 周四00:21写道：

现在是在窗口关闭的时候，发一个通知，不是很好，但是工作。

//------------------------------------------------------------
// NSWindowDelegate
func windowWillClose(notification: NSNotification) {
    NSNotificationCenter.defaultCenter()
        .postNotificationName(NOTIFY_ADV_CONF_CHANGED, object: nil)
}

—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
#44 (comment),
or mute the thread
https://github.com/notifications/unsubscribe-auth/AF8zUaUDI84GViMnBVUUBHGZxRpvfZ-jks5qjG-lgaJpZM4JquMZ
.

[qinyuhang]

话说这个travis ci怎么要了。。？点进去看还没有开始build？

[yichengchen]

上面的travis-ci从结果日志看起来貌似没什么问题……

[qinyuhang]

Travis CI 今天出故障了，现在OS X的build还没恢复。如何重启这个build？还是怎么办？

[meritozh]

push一个commit就能重启build了

[qinyuhang]

现在又来文件冲突了。。。？我只改了README里面的
我试着重新commit了一个新的，真心被Travis CI的故障坑了

[qinyuhang]

@qiuyuzhou 帮忙merge一下吧？谢谢。里面的文件冲突应该是README改了一些拼写错误。

[qiuyuzhou]

不好意思，最近比较忙。我尽快merge

秦宇航 notifications@github.com于2016年8月29日周一 下午4:33写道：

@qiuyuzhou https://github.com/qiuyuzhou
帮忙merge一下吧？谢谢。里面的文件冲突应该是README改了一些拼写错误。

—
You are receiving this because you were mentioned.

Reply to this email directly, view it on GitHub
#44 (comment),
or mute the thread
https://github.com/notifications/unsubscribe-auth/ACNATbGrqjBrURDiJNfiRwAJHaiB4fefks5qkplGgaJpZM4JquMZ
.

[qinyuhang]

好的，辛苦你啦

[qiuyuzhou]

@qinyuhang
我Review了你的代码，发现以下几个主要问题：

• 退出时修改了配置，再打开时shadowsocks是关闭状态
• 新加了个无用的proxy.pac文件在Git中
• “高级设置>Local PAC Listen Port:” 的输入框会有英文数字3位分割逗号

不佳的git实践：

• 没有将对Pods的修改和业务代码提交分开提交
• 将大量修改放到一个巨大提交中

[qinyuhang]

更新：“高级设置>Local PAC Listen Port:” 的输入框 我在run的时候没有看见分割逗号？（我第一次run的时候确实看见了，但是我记得改了右对齐之后这个已经被修正？）
～～～～～～～～～～～
受教了，那我是接下来rebase一下commit来解决pod库的问题吗？
此外退出时修改配置是为了避免干扰不需要FQ的情况，如果需要，我将它删掉。
proxy.pac 我会立刻删掉，放到下一个commit
“高级设置>Local PAC Listen Port:” 的输入框会有英文数字3位分割逗号 我会改正，放到下一个commit

[qiuyuzhou]

这次pods库的问题就算了。

[qinyuhang]

好的 其他的问题有需要继续修改的吗？

Qiu Yuzhou notifications@github.com于2016年9月1日 周四16:28写道：

这次pods库的问题就算了。

—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
#44 (comment),
or mute the thread
https://github.com/notifications/unsubscribe-auth/AF8zUQhrtIZkbCJlZDyZwr5-DC9OmtIqks5qloyfgaJpZM4JquMZ
.

[qiuyuzhou]

revert你后面那个 Fix typo提交，会造成conflict

[qinyuhang]

已revert。

Qiu Yuzhou notifications@github.com于2016年9月1日 周四16:32写道：

revert你后面那个 Fix typo提交，会造成conflict

—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
#44 (comment),
or mute the thread
https://github.com/notifications/unsubscribe-auth/AF8zUVTQvu85ViYJSenM1AVETbOSrPwqks5qlo2SgaJpZM4JquMZ
.

[yichengchen]

建议使用 webServer.startWithOptions(["BindToLocalhost":true,"Port":xxxx]) 来避免提示允许监听端口