htty icon indicating copy to clipboard operation
htty copied to clipboard

Check for and notify about updated gem on startup?

Open njonsson opened this issue 15 years ago • 6 comments

njonsson avatar Sep 30 '10 13:09 njonsson

I think this is a good idea, though the program should be polite and offer to disable the feature.

jgorset avatar Sep 30 '10 15:09 jgorset

Agreed.

njonsson avatar Sep 30 '10 15:09 njonsson

@njonsson have you thought about how to implement this? Would you really check at every startup that a new version has been published? Something like gem outdated | grep htty? Wouldn't that slow down too much the startup process?

gabrielelana avatar Mar 01 '14 07:03 gabrielelana

What I had in mind when I opened this issue was to execute a GET http://rubygems.org/api/v1/gems/htty.xml upon startup and to compare the value of XPATH /rubygem/version against HTTY::VERSION. Shelling out to gem outdated is a more loosely-coupled integration, but as you say it either introduces a startup delay or necessitates a background thread and attendant complexity.

I’m conflicted about this feature. While there’s a certain user-friendliness to an upgrade courtesy notice, it stands to reason that this kind of tool ought not to make HTTP requests without the user’s say-so.

Care to offer your opinion on the utility of this feature, @gabrielelana?

njonsson avatar Mar 04 '14 22:03 njonsson

I really don't know... Sure it will be nice but usually it's a feature implemented for long running processes, in background they could check for an update and in case warn the user, but in our case the user would pay the price for a check every time, it seems a little too much

Maybe we can check only sometimes, but this would mean to have some sort of file where to store the last check timestamp

Otherwise we could check before quit, as a user I'd rather pay at the end than at the start

@njonsson could we think about that a little more and pospone this enhancement for a later version?

gabrielelana avatar Mar 05 '14 17:03 gabrielelana

Agreed, @gabrielelana.

njonsson avatar Mar 18 '14 15:03 njonsson