I'll fix those myself and commit, as proposed by wraitii on IRC.
- Queries
- All Stories
- Search
- Advanced Search
- Transactions
- Transaction Logs
Advanced Search
Jun 30 2020
OK, everything good for me ??
Jun 27 2020
This looks good, I will test it later! ??
This looks good to me apart from the roundtrip change. Additionally I am fine with believing that SM uses UTF16 internally but what is the source of this affirmation? Is it still true with recent versions of SM?
Jun 26 2020
Jun 24 2020
This quote is both sexist by modern standards, and not about combat (even loosely). That is all. If there are other quotes with those two issues, we could definitely add them to the diff.
I think what was meant by Not a vital quote is that we are developing a strategy game of combat and thus quotes that are not related to war are not useful.
Jun 23 2020
I'm going to include it in D2563!
(Upon committing, I am only adding -o /dev/null to the cc call, in order not to generate an unneeded a.out file)
This works well on GCC 10 and clang 10.
Hi nephele, sorry for the delay in reacting. I was away finishing writing my PhD thesis. I am going to commit this fix. Thanks a lot for your work! ??
I am not sure I like this change... ? I feel like grouping the association between XML names and C++ classes in a single place is neater.
This looks fine, I can test it.
Jun 22 2020
The two new files should have the svn:eol-style property set to native like all other source files ??
May 27 2020
GunChleoc reports it doesn't correctly detect an old NVTT version.
May 24 2020
In D2766#116849, @Stan wrote:I mean that the proposed link has more chances to go down than GitHub :) https://github.com/boostorg/boost/releases but after a second look it seems they only provide the source there, not a preconfigured version so nvm I guess.
In D2766#116846, @Stan wrote:Isn't it safer to use the github mirror?
May 22 2020
In D2757#116558, @wraitii wrote:Haven't done the long-translation thing yet because I don't really understand what it is supposed to be doing.
This sounds like a good idea ?
May 21 2020
Update after rP23686, so that the mac slave builds the new version.
I am going to commit this, but with a suffix for spidermonkey, as well as using numbers (instead of tentative dates) for the suffixes.
May 19 2020
Make the preprocessor change minimal.
May 17 2020
Thanks! I'll try to commit this during the week, we have some bank holidays...
And one final idea: maybe the nvtt version should have a wildfiregames suffix so that it gets rebuilt when we add some patches, like I will do in D2563.
Additionally I had this idea: P203
Hi Krinkle, very nice patch! It will help the CI considerably.
May 9 2020
Apr 1 2020
Mar 31 2020
I will commit this today. Thanks adrian for the patch and s0600204 for the help!
Mar 28 2020
In D2671#112871, @nephele wrote:Else at the very least I'd prefer to test the need for execinfo once at the beginning, so that we don't have calls to the compiler in the middle of our (already complex) prebuild logic.
Why would it matter?
I see. In that case I suppose this is OK (I find it rather ugly but that's personal taste).
Hi! Thanks for your contribution.
Thanks for the rebase @s0600204! Jenkins is indeed broken when source files are deleted (or moved) but there is still a build error on a clean build due to the test file.
Mar 9 2020
I see! Thanks for the info. I'll include your changes in the other patch then 👍
Hello, thanks for the contribution! This should be already fixed by D2523, which is not yet committed. Could you take a look and check that it works? Else I'll update that other diff.
Mar 5 2020
Mar 1 2020
In D2563#111168, @Stan wrote:Ricotz told today on IRC it still doesn't work on ARM64. See https://launchpadlibrarian.net/467121489/buildlog_ubuntu-bionic-arm64.0ad_0.0.24~r23516-0ubuntu1~18.04~wfg1_BUILDING.txt.gz
Feb 17 2020
Feb 16 2020
Do you plan to run the tests manually, or would you be interested in having Jenkins run the tests when a patch touches source/tools/lobbybots?
Feb 12 2020
I will test this and hopefully commit it during the upcoming weekend. I am a bit short on free time these days, so unfortunately when things are related to libraries some of the steps include waiting for me. Don't hesitate to contribute to other areas of the code in the meantime. Or if you want you can start working on the future cppformat upgrade steps since they seem to involve a lot of work.
Feb 10 2020
(you also needed to specify the repository in order to trigger a build, I fixed that)
Feb 9 2020
In D2613#110077, @adrian wrote:I propose to merge this changes into codebase (I can restore old cppformat name) and create another ticket to update fmt to version 6.x. Unfortunately I'm not sure how much time I will have in the future, so I think better option is to not "lock" this issue further.
I agree, and actually I think it's very nice to use this small upgrade to rename the lib, so you can leave it like this.
Feb 4 2020
Feb 3 2020
Feb 1 2020
Just a note since I see the patch only now:
Jan 13 2020
Can you attach the dds generated by nvtt that it doesn't manage to load? Thanks :)
Hey! This is already in D2523.
In D2563#107181, @Stan wrote:This fixes NVTT build however I still get for some images:
Jan 12 2020
Remove the macOS change that is out of place and is likely to create issues.
@Angen I forgot to close the diff in rP23382 ?
and check that the new va_list initializer works on other compilers
In D2548#107040, @Stan wrote:@Itms can you close it?
Jan 10 2020
In D2557#106874, @elexis wrote:(A commit reference would be useful to ease commit traversal while reviewing, auditing or revisioning)
It was rP16824. And that is funny: looking at the commit, I had written a TODO, so I knew! I then removed the TODO in rP16827, just three revisions later, when I thought the improvements I made were enough.
Excluding the in-game terrain updates, which are not actually implemented, everything works fine for me and the code looks good.
In D2557#106789, @vladislavbelov wrote:I think it's not enough, because it has only water level changes. That means the whole grid will be updated (no partial update).
Hey, this looks great! I didn't realize the message had that terrain information when I wrote the code!
Jan 7 2020
In rP23305#40462, @Krinkle wrote:Also, I was expecting the script to track the version of lib it built instead of boolean. Seems simple to implement, will consider that later.
Ideally we would do the same as for SpiderMonkey (see libraries/source/spidermonkey/build.sh):
if [ -e .already-built -a .already-built -nt README.txt ]
so that changes to the README trigger a rebuild. However, this is bypassed (including for SM) by build-osx-libs.sh. There is a big discrepancy between macOS and other unixes with respect to the process followed to build bundled libs.
Jan 6 2020
rm -f libraries/source/nvtt/.already-built
in order to actually build the new nvtt version with build-osx-libs.sh.
Jan 3 2020
Dec 31 2019
In D2491#105479, @elexis wrote:In D2491#104180, @Itms wrote:There is already this PR by a friend
While I appreciate the work on the software, calling leper your friend is a punch in the face and I will not go down in history as someone who read it and left it uncommented.
Also builds and runs --without-pch on Linux (gcc 7). ?
Dec 30 2019
This is good for me! I think you can commit this just after I commit D2524.
Tests fixed.
Thanks again for the patch Inari. I am commandeering it to fix the tests as well. I will commit this just after D2524.
In D1380#79218, @wraitii wrote:Right now our precompiled headers also act as global include files, when they probably shouldn't.
Thanks for the patch and I owe you a beer or something for the delay ? The changes look perfect. Let's get them in before the copyright year becomes wrong...