Latest Posts

Topic: Adding updates to launchpad

Tinker
Avatar
Joined: 2014-09-12, 10:53
Posts: 183
Ranking
Widelands-Forum-Junkie
Location: DDR
Posted at: 2017-11-15, 12:33

I think I have sorted out what I am doing with bazaar. As a test I have created an update to starting conditions which appears to be revision 8490. I have applied for membership of the dev team. I am now waiting approval and to see if my changes are acceptable.

The starting conditions I have submitted are simple, a new starting condition for each tribe, I also have one for Frisians to either add later or add to the Frisians branch. My thought behind the new start is that the headquarters is most used but boring after a while, trading post and fortified village are to easy, both a sort of cheat mode. I have added a Village start that can be used on maps where the description says you have been living in the area for a while, you get a few basic buildings, nothing that needs thought on placing, to offset this you have lower military. The start condition has been tested on most maps with human and AI.


The only way to prevent a mass extinction event is to remove homo sapiens, or stupiditus, from the planet. It may be to late for this final solution though.

Top Quote
Tino
Joined: 2009-02-20, 17:05
Posts: 250
Ranking
Tribe Member
Location: Somewhere in Germany...
Posted at: 2017-11-15, 13:33

Hi,

you don't need to be a member of the widelands team to publish your code and make a merge request.

Only if you want to be able to merge code to trunk directly or to publish branches owned by the team.


Top Quote
Tinker
Avatar
Joined: 2014-09-12, 10:53
Posts: 183
Ranking
Widelands-Forum-Junkie
Location: DDR
Posted at: 2017-11-15, 17:11

Well I created a branch and tried to push it but was told I could not as I was not a member of the dev team, My application was thrown out as I have not added any code, So how do I publish my contribution for approval?


The only way to prevent a mass extinction event is to remove homo sapiens, or stupiditus, from the planet. It may be to late for this final solution though.

Top Quote
kaputtnik
Avatar
Joined: 2013-02-18, 20:48
Posts: 1627
Ranking
One Elder of Players
Location: Germany
Posted at: 2017-11-15, 17:49

Try this command (change 'YourUsername' with your Launchpad-ID). The BRANCHNAME is usually a short discription of the proposed change, eg. new_starting_conditions.

bzr push lp:~YourUsername/widelands/BRANCHNAME

After successful uploading the branch you can click on it and 'Propose for merging'.


Top Quote
Tinker
Avatar
Joined: 2014-09-12, 10:53
Posts: 183
Ranking
Widelands-Forum-Junkie
Location: DDR
Posted at: 2017-11-15, 22:55

That was what I tried, I get this error ssh_askpass: exec(/usr/lib/ssh/ssh-askpass): No such file or directory bzr: ERROR: Connection closed: Unexpected end of message.

This looks like a permissions error although I have a registered key.


The only way to prevent a mass extinction event is to remove homo sapiens, or stupiditus, from the planet. It may be to late for this final solution though.

Top Quote
GunChleoc
Avatar
Joined: 2013-10-07, 15:56
Posts: 2715
Ranking
One Elder of Players
Location: RenderedRect
Posted at: 2017-11-16, 07:49

Sorry, I hadn't seen this thread when I threw out your application, and your username at launchpad is radically different, so I had no idea it was you.

You can still push though if you replace widelands-dev with ~username

Have you registered an ssh key with Launchpad? https://help.launchpad.net/YourAccount/CreatingAnSSHKeyPair

Edited: 2017-11-16, 07:49

Busy indexing nil values

Top Quote
Tinker
Avatar
Joined: 2014-09-12, 10:53
Posts: 183
Ranking
Widelands-Forum-Junkie
Location: DDR
Posted at: 2017-11-16, 10:22

Ah username, I live in several different compartments and they are radically different, perhps I should have mentioned that.

I got it to work today, perhaps hysterisis in the key system? In any case the branch is there and I have put it up for review. I have some more complex changes in the pipline now that I know how to upload.


The only way to prevent a mass extinction event is to remove homo sapiens, or stupiditus, from the planet. It may be to late for this final solution though.

Top Quote
GunChleoc
Avatar
Joined: 2013-10-07, 15:56
Posts: 2715
Ranking
One Elder of Players
Location: RenderedRect
Posted at: 2017-11-17, 08:19

Thanks for the branch - I have left some comments in the code review.


Busy indexing nil values

Top Quote
Tinker
Avatar
Joined: 2014-09-12, 10:53
Posts: 183
Ranking
Widelands-Forum-Junkie
Location: DDR
Posted at: 2017-11-17, 10:58

I took me a while to find out where the code review was, sorted now, and comments added.


The only way to prevent a mass extinction event is to remove homo sapiens, or stupiditus, from the planet. It may be to late for this final solution though.

Top Quote