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

Syncthing icon for Windows binaries. #3668

Closed
damajor opened this issue Oct 13, 2016 · 10 comments
Closed

Syncthing icon for Windows binaries. #3668

damajor opened this issue Oct 13, 2016 · 10 comments
Labels
enhancement New features or improvements of some kind, as opposed to a problem (bug) frozen-due-to-age Issues closed and untouched for a long time, together with being locked for discussion good first issue Good starting points for new contributors

Comments

@damajor
Copy link
Contributor

damajor commented Oct 13, 2016

Hi,

I just noticed that there is no SyncThing icon to the Windows binary.
Will it be added ?

@AudriusButkevicius
Copy link
Member

If you want to contribute additions to the build script, sure why not.

@AudriusButkevicius AudriusButkevicius added the enhancement New features or improvements of some kind, as opposed to a problem (bug) label Oct 13, 2016
@damajor
Copy link
Contributor Author

damajor commented Oct 13, 2016

I'll try to look into this new thing for me :)

@damajor
Copy link
Contributor Author

damajor commented Oct 13, 2016

I look quickly for a way to do that and it appears that there is no easy solutions.

  • use some kind of resource patcher to embed the icon, I do not recommend this method as it requires external executable to be run and patch the SyncThing binary.
  • use rsrc https://github.com/akavel/rsrc, it is pretty similar to the other but generate a resource used by the Go compiler at link time when it bundles all binaries into a single executable.
  • use https://github.com/josephspurrier/goversioninfo, it is almost the same as the previous one but include the full structure used by Windows to show file details when you show its properties.

The last two seems good, but I dislike the fact that it is not portable across operating systems...

So you can leave the ticket open for tracking purpose, but at this time I think it is too much work for a such little enhancement.

@AudriusButkevicius
Copy link
Member

What makes you think they are not portable? I think you can run them on any OS as long as you are targeting a Windows binary.

@damajor
Copy link
Contributor Author

damajor commented Oct 14, 2016

You are right. The last two should be portable.

@calmh
Copy link
Member

calmh commented Nov 23, 2016

That said, is there a point to having a nice icon when double clicking it just results in an ugly terminal window? This feels to me like something that should be solved by a wrapper or a hypothetical future version that actually has a GUI...

@AudriusButkevicius
Copy link
Member

It's a low hanging fruit. Clicking on an icon that looks like a .com file is much worse, despite the fact it opens an ugly window.

@calmh calmh added this to the Unplanned (Contributions Welcome) milestone Jan 23, 2017
@calmh calmh added the good first issue Good starting points for new contributors label Jan 23, 2017
@alex-ilin
Copy link

I run Syncthing occasionally, when I backup my phone to my laptop once a week. I run it from the Total Commander toolbar. It would be nice to at the very least have an ico-file somewhere in the Syncthing folder so I could manually associate it with the toolbar button.

An icon embedded in the exe would be a convenience, but it can be seen as a second step.

@AudriusButkevicius
Copy link
Member

The ico files are available on the github repo.

@calmh calmh removed this from the Unplanned (Contributions Welcome) milestone Feb 11, 2018
@calmh
Copy link
Member

calmh commented Nov 1, 2018

Implemented

@calmh calmh closed this as completed Nov 1, 2018
@st-review st-review added the frozen-due-to-age Issues closed and untouched for a long time, together with being locked for discussion label Nov 2, 2019
@syncthing syncthing locked and limited conversation to collaborators Nov 2, 2019
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
enhancement New features or improvements of some kind, as opposed to a problem (bug) frozen-due-to-age Issues closed and untouched for a long time, together with being locked for discussion good first issue Good starting points for new contributors
Projects
None yet
Development

No branches or pull requests

5 participants