RE: [Patch v4] Multisession support plus Leading Login support (pull request)
Mike Christie wrote: >Merged all the patches. They should show up in a couple minutes on the >kernel.org git tree. Great, thanks! >Thanks for your work on this and for putting up with the coding style junk. I >hate to be the-coding-style-comment-guy. No worries, I completely understand. -- Jim Ramsay -- You received this message because you are subscribed to the Google Groups "open-iscsi" group. To post to this group, send email to open-iscsi@googlegroups.com. To unsubscribe from this group, send email to open-iscsi+unsubscr...@googlegroups.com. For more options, visit this group at http://groups.google.com/group/open-iscsi?hl=en.
Re: [Patch v4] Multisession support plus Leading Login support (pull request)
On 07/07/2011 07:42 AM, Jim Ramsay wrote: > Mike Christie wrote: >> Was v3 supposed to have the coding style fixups? >> >> You can catch most of them by running the patches over the kernel's >> checkpatch.pl script (I attached it here). I have this setup so it is > run when I am >> merging a patch. It is still spitting out errors for your set. You can > ignore the >> _func_ one, but there are some 80 cols and some spacing ones. > > I was confused for a while since the only other checkpatch.pl issue I > was seeing in my 4-patch series was another minor 80-char overrun, until > I also looked at my other submission, the "Leading Login" support most > recently posted as a "Patch v2" in a separate thread. The leading-login > patch did indeed have a couple more 80-char infractions and some other > spacing errors, though in my defense all the (* login_fn) were cut&paste > from existing code so I had thought they would also be safe to ignore. I did not check that patch yet when I wrote the mail above. They were ok to ignore. It is fine though. > > So I've re-cleaned this leading-login patch and the multisession series. > The only checkpatch.pl warning now is the __func__ one. > > And rather than fighting with my SMTP server again, I've just pushed > everything here: > git://repo.or.cz/open-iscsi/multisession.git > Merged all the patches. They should show up in a couple minutes on the kernel.org git tree. Thanks for your work on this and for putting up with the coding style junk. I hate to be the-coding-style-comment-guy. -- You received this message because you are subscribed to the Google Groups "open-iscsi" group. To post to this group, send email to open-iscsi@googlegroups.com. To unsubscribe from this group, send email to open-iscsi+unsubscr...@googlegroups.com. For more options, visit this group at http://groups.google.com/group/open-iscsi?hl=en.
[Patch v4] Multisession support plus Leading Login support (pull request)
Mike Christie wrote: >Was v3 supposed to have the coding style fixups? > >You can catch most of them by running the patches over the kernel's >checkpatch.pl script (I attached it here). I have this setup so it is run when I am >merging a patch. It is still spitting out errors for your set. You can ignore the >_func_ one, but there are some 80 cols and some spacing ones. I was confused for a while since the only other checkpatch.pl issue I was seeing in my 4-patch series was another minor 80-char overrun, until I also looked at my other submission, the "Leading Login" support most recently posted as a "Patch v2" in a separate thread. The leading-login patch did indeed have a couple more 80-char infractions and some other spacing errors, though in my defense all the (* login_fn) were cut&paste from existing code so I had thought they would also be safe to ignore. So I've re-cleaned this leading-login patch and the multisession series. The only checkpatch.pl warning now is the __func__ one. And rather than fighting with my SMTP server again, I've just pushed everything here: git://repo.or.cz/open-iscsi/multisession.git Please pull and merge the tag 'submitted/leading_login_v4' (d6c2a0fec4ea17d09d70ed24ecc6886590441c6a) -- Jim Ramsay -- You received this message because you are subscribed to the Google Groups "open-iscsi" group. To post to this group, send email to open-iscsi@googlegroups.com. To unsubscribe from this group, send email to open-iscsi+unsubscr...@googlegroups.com. For more options, visit this group at http://groups.google.com/group/open-iscsi?hl=en.