Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

bitcoin-tx.exe isn't shipped as part of the NSIS installers on Windows #5605

Closed
Michagogo opened this issue Jan 6, 2015 · 10 comments
Closed

Comments

@Michagogo
Copy link
Contributor

No description provided.

@Michagogo Michagogo changed the title bitcoin-tx.exe isn't shipped as part of the NSIS installer on Windows bitcoin-tx.exe isn't shipped as part of the NSIS installers on Windows Jan 6, 2015
@laanwj laanwj added this to the 0.10.0 milestone Jan 6, 2015
@laanwj
Copy link
Member

laanwj commented Jan 6, 2015

Ping @theuni

Maybe we need a discussion about what to in include in the installer versus the .zip. I'd imagine that by far most people that download the installer only use bitcoin-qt. This would be analogous to the default download (.dmg) for MacOSX, which always only included the graphical client.

@jgarzik
Copy link
Contributor

jgarzik commented Jan 6, 2015

+1 @laanwj

I would prefer to keep the installer-shipped contents consistent across platforms.

@luke-jr
Copy link
Member

luke-jr commented Jan 6, 2015

NSIS supports a nice "select what to install" step that would be appropriate IMO.

@Michagogo
Copy link
Contributor Author

I can't disagree with consistency. The question is, would it be better to add the other stuff to OS X, or remove it from Windows? The thing is that the two systems have very different install paradigms. OS X is "drag (what looks like) this one file/blob into place", while Windows is "run this installer, it puts everything in the right place, updates it if needed, and registers with the system". I would personally think that adding it to OS X (if that's possible, I don't know much about OS X's apps-that-are-really-directories-but-pretend-to-be-files-in-Finder-and-magically-start-the-right-executable structure) would be preferable to removing them from Windows, because for Windows users like myself, having the exe be in a specific place and auto-updated is nice, and I assume that it would be the same for OS X users, with the ability to have a single update action that updates everything in place, with all paths remaining as-is. So from that standpoint, I think I would vote for adding the extras to OS X rather than dropping them from Windows. But I don't know if this kind of change is too big to go in between RCs. If so, I would say that we should add bitcoin-tx.exe for now, sticking with the current practice of shipping all the bins (other than the tests) in the installer.

@Michagogo
Copy link
Contributor Author

Oh, that's a good point -- why aren't we using the feature selector?

@theuni
Copy link
Member

theuni commented Jan 7, 2015

Either way is fine by me. For OSX, we could add a "utils" folder inside the dmg, although there's no obvious/standard way to install them. Instead, the user would just keep the dmg around, or drag the folder somewhere.

If we went that route, we could add the libs and includes as well. And do the same for windows, with options for each. In doing so, we'd negate the need for separate tarballs/zips.

Sounds very reasonable to me, though I'd much prefer to wait until after 0.10 to do it.

@Michagogo
Copy link
Contributor Author

Is it considered a bad thing to have other executables inside a .app
bundle/directory? I was thinking they could go in there, picturing it as
"somewhere out of the way, where a slightly more advanced user can look if
they want to, that won't be in the way and confusing to more basic users".
But I don't actively use a Mac nor am I familiar with its development
practices, so there may be a good reason not to. I don't know.

On Wednesday, January 7, 2015, Cory Fields notifications@github.com wrote:

Either way is fine by me. For OSX, we could add a "utils" folder inside
the dmg, although there's no obvious/standard way to install them. Instead,
the user would just keep the dmg around, or drag the folder somewhere.

If we went that route, we could add the libs and includes as well. And do
the same for windows, with options for each. In doing so, we'd negate the
need for separate tarballs/zips.

Sounds very reasonable to me, though I'd much prefer to wait until after
0.10 to do it.


Reply to this email directly or view it on GitHub
#5605 (comment).

@laanwj
Copy link
Member

laanwj commented Jan 7, 2015

What I would like to avoid is including more than necessary, increasing the sizes of the popular downloads. Due to our reliance on static linking, every additional executable, even simple utilities add a lot of weight.

In all the time we've shipped MacOSX releases, almost one ever complained about there not being a separate bitcoind/bitcoin-cli. The few people I know that need them (such as @rktoomey) prefer them as separate executables in a .zip, because they can be easier integrated into scripting/testing workflows. So I'd argue against including them in the dmg. 99% of users won't even realize they are there and they're just dead-weight.

I'm not against adding bitcoin-tx to the Windows installer, but my experience is that most windows users won't be using command line utilities, because they prefer clicking but also because the windows cmd.exe is terrible to use as a shell.

@theuni
Copy link
Member

theuni commented Jan 8, 2015

As a data point: For OSX, adding bitcoind, bitcoin-cli, and bitcoin-tx to the dmg adds ~5 megs to the total size. 11Mb -> 16Mb. The 0.9.3 dmg is already 16Mb, due to the shared Qt.

I still really don't care either way, just providing real numbers.

@laanwj laanwj removed this from the 0.10.0 milestone Jan 12, 2015
@laanwj
Copy link
Member

laanwj commented Feb 16, 2016

Closing this, "works as intended". If you need the utilities, there is the zip archive which has them all.

@laanwj laanwj closed this as completed Feb 16, 2016
@bitcoin bitcoin locked as resolved and limited conversation to collaborators Sep 8, 2021
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

No branches or pull requests

5 participants