Proposed fix for issue 330115

[Shantanu Tushar Jha]

On Fri, Jan 24, 2014 at 7:09 PM, R.Harish Navnit <harishnavnit at gmail.com>wrote:

> On Fri, Jan 24, 2014 at 6:19 PM, Sinny Kumari <ksinny at gmail.com> wrote:
>
>> This link is not what you wanted to paste ;)
>>
>
> Oops ! I'm sorry. This is what it was meant to be : http://goo.gl/XLAz2p
>
> Moreover , now I'm not getting the error log that I was getting earlier.
> It was a little bit different
>
>> Also, send diff of code by using "git  diff". It will be easier to llok
>> into issue :)
>>
>
> git diff doesn't really do much : http://goo.gl/NMtN0j  , although I've
> added and committed changes.
>

She was assuming you haven't committed the changes. git diff by default
will only show uncommited the changes. To show the diff of a particular
commit use "git diff SHA" where SHA is the commit-id of the particular
commit. A shortcut for showing the diff of your last commit is "git diff
HEAD", as HEAD means the most recent commit.

Also, you cannot push directly to the repository on KDE's git unless you
have developer access. The process anyway is to create the diff, save it to
a file, upload it to https://git.reviewboard.kde.org/ where one of us will
review it (please add "Plasma" to groups when creating the review request).


>
> Regards.
>
> Thanking You,
> R.Harish Navnit
> The Enigma <http://harishnavnit.wordpress.com>
>
> _______________________________________________
> Plasma-devel mailing list
> Plasma-devel at kde.org
> https://mail.kde.org/mailman/listinfo/plasma-devel
>
>


-- 
Shantanu Tushar    (UTC +0530)
http://www.shantanutushar.com
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/plasma-devel/attachments/20140124/869fd681/attachment-0001.html>