- Queries
- All Stories
- Search
- Advanced Search
Advanced Search
Oct 5 2019
Sep 27 2019
Sep 26 2019
Sep 25 2019
Sep 24 2019
In D24149#536863, @valeriymalov wrote:Should I land it on master or it seems good enough for the stable branch?
Sep 23 2019
Sep 22 2019
Ping? Any chance to get updated patch with addressed review comments?
Sep 8 2019
Sep 7 2019
Sep 1 2019
Aug 29 2019
Aug 22 2019
Aug 21 2019
Aug 13 2019
Aug 12 2019
Aug 8 2019
In D21903#508769, @aferraris wrote:Finally, this patchset has been thoroughly tested for ext4 and btrfs filesystems.
If everything is OK, I can add support for fat32 blocksize before merging.
Aug 4 2019
Aug 2 2019
Yes, and you need to add it to cmake. That part of code is autogenerated.
Before it's pushed (once it compiles) also refactor code a bit so that
there isn't that much code duplication.
Jul 22 2019
I guesss it's alright to go in for now.
Jul 21 2019
Jul 13 2019
Looks alright. I'll try to compile myself and see what are those linking errors when you have those two lines uncommented...
Jul 11 2019
Jul 8 2019
-i bytes-per-inodeI am a bit worried that this API would be hard to use from GUI clients. Although in general, I think direction is good.
Jun 24 2019
Jun 22 2019
Jun 15 2019
I think QSignalSpy is in QtTest, not QtWidgets.
Jun 9 2019
So far rest seems alright.
Can you also mention motivation in commit message
Jun 8 2019
Once issues mentioned above are fixed.
Oh, I see. Sorry for missing that out. Then I guess commit it to master.
If you want this commit, you can just merge master branch into your branch.
Jun 7 2019
Looks good, just keep space consistent where I mentioned. Then you can push this to master branch.
I think startHelper() is much better.
Jun 6 2019
Jun 5 2019
Jun 3 2019
Jun 2 2019
Jun 1 2019
Hmm, regarding license, I guess it will be fine. KTorrent is GPLv2 or later. So GPLv3 would work.
Yes, it looks fine to me too. Please fix those issues that wbauer already mentioned and then you can commit (or ask somebody to commit if for you if you don't have commit access).
May 31 2019
May 30 2019
May 25 2019
May 24 2019
In D21340#469511, @cjlcarvalho wrote:There are some procedures missing in this test. First, you need to look if mdadm is installed in the system before testing RAID. Then, you need to test the creation of a RAID device and check if it was successfully created. After that, you can try to create partitions on it and test if it exists.
May 23 2019
May 21 2019
Maybe push to 4.0 branch?
May 19 2019
May 18 2019
May 17 2019
Also in general, I think we should create a bool variable where we store data, so that we only run this
Well, in principle both mean exactly the same thing, so I don't have strong opinion on this. I probably use them interchangeably anyway. Well, it's auto, so we don't care what type it is... So based on that I would say no need to make a commit. Other parts of KDE codebase also don't have unified style here. Just checked kwin, it has both auto and auto *
In D21251#466412, @shubham wrote:Yesterday I tried for putting theses revisions over gitLab. but could not succeed. I tried doing git push origin master and it said access denied. So what should I do now to push it remote and the merge? Thanks
In D21251#466408, @shubham wrote:I have commit access, but It says access denied. Remember you had said it is hosted on remote server.
In D21251#466401, @shubham wrote:Oh, I do by running kdesrc-build script, which takes care of removing old build directory.
Please push this and the other revision on my behalf : )
oh ok, that's fine then.
In D21251#466395, @shubham wrote:please test that it builds in a clean build dir.
I do not know what do you mean? tests?