In D8831#168209, @kfunk wrote:Thanks a lot for these patches!
- Queries
- All Stories
- Search
- Advanced Search
Feed Advanced Search
Advanced Search
Advanced Search
Nov 16 2017
Nov 16 2017
geetamc added a comment to D8831: gcclikecompiler: Fix execution of compiler qprocess under windows.
Nov 15 2017
Nov 15 2017
geetamc added a comment to D8831: gcclikecompiler: Fix execution of compiler qprocess under windows.
So the actual error is: gcc.exe: error: \\.\NUL: Invalid argument
geetamc added a comment to D8831: gcclikecompiler: Fix execution of compiler qprocess under windows.
In D8831#168189, @brauch wrote:I understand what the patch does, but can you explain why this fixes your issue?
Nov 14 2017
Nov 14 2017
- remove unnecessary ifdef block
Nov 13 2017
Nov 13 2017
Nov 10 2017
Nov 10 2017
Nov 8 2017
Nov 8 2017
Oct 28 2017
Oct 28 2017
In D8523#161270, @kfunk wrote:@geetamc Note: Please fixup your user name to in Git (git config user.name).
I needed to amend your patch in order to fix this issue:
remote: Audit failure - Commit 55f2f00c8376837c9084b36413ae45b3f5652569 - Non-full name: Geetam remote: FATAL: VREF/kde-update-checks: helper program exit status 256 remote: Push declined - commits failed auditThanks for the patch!
- fixed weird indentation
Oct 27 2017
Oct 27 2017
geetamc added a reviewer for D8523: Use native separators while copying filename to clipboard: KDevelop.
Jul 25 2017
Jul 25 2017
Jul 13 2017
Jul 13 2017
In D6689#125180, @kfunk wrote:Thanks for the patches, Geetam.
I encourage you to start using Arcanist to upload the patches as that will a) make reviewing the patches easier (because there's "context" in the diff view), b) add proper author information to the Phab Diff.
Please see: https://community.kde.org/Infrastructure/Phabricator#Using_Arcanist
Jul 11 2017
Jul 11 2017
Jul 4 2017
Jul 4 2017
geetamc updated the test plan for D6505: Fix external launch of executables with spaces in paths[windows].
geetamc updated the summary of D6505: Fix external launch of executables with spaces in paths[windows].
May 16 2017
May 16 2017
geetamc updated the summary of D5777: Fix splitting of root installation command, add default value for the same in .kcfg.
geetamc added a comment to D5777: Fix splitting of root installation command, add default value for the same in .kcfg.
In D5777#109999, @apol wrote:I don't really understand the patch, there's the bit there so that if (cmd.isEmpty) kdesu -t. What are you trying to fix?
May 8 2017
May 8 2017
geetamc retitled D5777: Fix splitting of root installation command, add default value for the same in .kcfg from Fix check for empty su command for installing as root to Fix splitting of root installation command, add default value for the same in .kcfg.
geetamc updated the diff for D5777: Fix splitting of root installation command, add default value for the same in .kcfg.
used KShell:splitArgs to split su command.
geetamc added a comment to D5777: Fix splitting of root installation command, add default value for the same in .kcfg.
@apol Thanks, I will surely.
But I will update the revision right now without the change as I accidentally created the diff against an older base.
Ignore the previous update, I am making a new review request.
-Added a default value for entry "suCommand" in makebuilderconfig.kcfg
-The qstring suCommand is checked for emptiness instead of qstringlist "suCommandWithArg".
(Reason: when an empty qstring is split, the returned qstringlist.isEmpty() is False)
May 7 2017
May 7 2017
Added a comment explaining the need of "if(suCommandIsDigit)" block.
In D5122#107479, @zhigalin wrote:Hello and thanks for your effort.
Just a few considerations:
- When you edit an UI file, make sure not to forget to increase it's version
- You also forgot to bind kcfg_suCommand.enabled(bool) to kcfg_installAsRoot.toggled(bool)
- You must use QStringLiteral, not just plain strings
- Try placing less newlines around
May 5 2017
May 5 2017
geetamc added a comment to D5719: Use https://www.google.com instead of http://www.google.de in google selection external script.
In D5719#107141, @brauch wrote:Yes, makes sense. Do you have commit access?
Apr 27 2017
Apr 27 2017
• hectordavid awarded D5122: Allow users to install via a custom root installation command a Burninate token.
Mar 31 2017
Mar 31 2017
In D5122#99210, @ltoscano wrote:Few suggestions related to the commit message: it should follow the guidelines for git message.
So the first line should be a short summary of the change (no need to mention the bug), "allow users to install via custom command" or something like that.The bug should report in one of the other line not as link but with
BUG: nnnnnn
geetamc retitled D5122: Allow users to install via a custom root installation command from This patch fixes bug 374144, Basically it allows users to install via a custom root installation command to Allow users to install via a custom root installation command.
Mar 30 2017
Mar 30 2017
If entry "Su Command" is an integer in range [0, 3) (which can be a scenario when user, for the first time opens his/her project in the updated kdevelop)
then suCommand is set appropriately and is written to entry "Su Command" .
Alright so I feel, regarding my last comment, Having an extra variable suCommandStr and an extra readEntry() is unnecessary, In the switch block I will simply assign suCommand the appropriate value, and write suCommand to the "Su Command" Entry.
Also I feel updating the diff will give you guys a better Idea so I will do the same in a while.
@ematirov Thank you for reviewing! Have a look at the inline comment.
Mar 22 2017
Mar 22 2017
geetamc updated the summary of D5122: Allow users to install via a custom root installation command.
arguments such as "-t" now pass correctly.
geetamc updated the summary of D5122: Allow users to install via a custom root installation command.
I believe that having combo box editable is a cleaner way to solve the same problem, I will revert the changes if reviewers feel that having a plain text edit and an entry for Custom Command in combo box is more appropriate.
Mar 21 2017
Mar 21 2017
I forgot to implement a feature mentioned by brauch, will do it tomorrow.
Since I am new to submitting patches would you please tell me when updating diff should I upload the diff representing the latest changes or one that represents the whole patch?
The build was successful, the patch now works as intended please let me know if I missed something.
In D5122#96551, @kfunk wrote:In D5122#96545, @geetamc wrote:My build fails right now after a git pull
kdevelop/languages/clang/duchain/macronavigationcontext.cpp:58:28: error: ‘prefix’ was not declared in this scopeYou need to git pull in kdevplatform and recompile/reinstall.
I have made most of the changes also I have an exam tomorrow so I will definitely update the diff by tomorrow.
Thank you both brauch and kfunk for reviewing.
-Corrected the casing of "Custom Command"
-Appended ':' to the label
-Label and text edit of Custom Command are now disabled by default
-removed unnecessary 'else'
My build fails right now after a git pull
In D5122#96534, @brauch wrote:Can you make the line edit disabled when something else than "custom command" is selected?
Otherwise, LGTM, thanks for the patch!
geetamc updated the summary of D5122: Allow users to install via a custom root installation command.