-
Notifications
You must be signed in to change notification settings - Fork 49
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
Is there a new maintainer? #71
Comments
There's no new maintainer @qirtaiba and you'd be more than welcome to take that on. Let me know if you're interested and I can add you as a collaborator! |
Sure, thanks. |
@simonv3 I am interested in maintaining quick-survey. I looked at this last night. It seems that it's going to take some work on the Sandstorm side of things before we can make a release. Are you willing to work with me to make a new release? |
@troyjfarrell I am, though my availability is pretty limited. Ideally there'll be a way to transfer that ability completely to you. |
We can do that. I'll send a message to sandstorm-dev to see what the proper procedure is. (I recall reading something about me generating signing keys and the Sandstorm devs adding some code to Sandstorm, so it expects the new keys.) |
There are two ways to handle it. Either Simon can extract the app signing key for Quick Survey from his Sandstorm keyring and give it to you, so you can sign the app with no changes to Sandstorm, or you can release it under a new appId, and Kenton can add a replacement key entry into Sandstorm so that it will treat it as an update to the original app. Either way, you should probably change the author metadata, so it uses your name instead of his, but that's separate from the key that signs the app. We can definitely help you through this process if you get the app updated and need any help publishing the updated version. (I am currently the person who pushes the approval button on app updates.) |
@simonv3 Do you still have your app signing key? (It would likely be in a file in a .sandstorm folder of your home directory, in a file called sandstorm-keyring.) If replacing the appId is necessary, @troyjfarrell may find this file pretty self-explanatory (or self-documenting, I guess) on what's involved: https://github.com/sandstorm-io/sandstorm/blob/master/src/sandstorm/appid-replacements.capnp (Apparently, according to that file, you actually don't change the appId in the pkgdef when you rekey it, which I didn't know.) |
I’ll get to work on a new release. (I’d like to update the dependencies.) |
@ocdtrekkie I'm fairly sure I don't have the key anymore. I've gotten a new computer recently and that didn't make it across, sorry all :( |
Thanks for the link. I don't mind going through the key replacement process. It looks like we'll have to play some identity games, presumable to verify that Simon is still the same Simon who is willing to hand over maintainership:
It's good timing. I've been wanting to generate a new PGP key for a while. I'll do that as part of this process. |
I'm OK with simon just approving the PR to appid-replacements.capnp. |
Sounds good. I'm working on automating the build process. I suspect that I'll be ready to spin a new release around the beginning of April. |
@troyjfarrell Stumbling around and figured I'd ask if you were still interested in this, as we are working on https://sandstorm.io/news/2020-02-03-reviving-sandstorm and hence interested in reaching out to people contributing to Sandstorm and/or Sandstorm apps. |
There are a bunch of forks of this project, but I'm not sure if any of them is working on fixing bugs like #54. I might be interested in becoming the new maintainer if nobody else is going to do that? These don't seem like super complex bugs to fix, or to pay someone a bug bounty to fix.
The text was updated successfully, but these errors were encountered: