Return-Path: X-Original-To: apmail-manifoldcf-user-archive@www.apache.org Delivered-To: apmail-manifoldcf-user-archive@www.apache.org Received: from mail.apache.org (hermes.apache.org [140.211.11.3]) by minotaur.apache.org (Postfix) with SMTP id D931E1090C for ; Fri, 22 Nov 2013 20:47:46 +0000 (UTC) Received: (qmail 91664 invoked by uid 500); 22 Nov 2013 20:47:46 -0000 Delivered-To: apmail-manifoldcf-user-archive@manifoldcf.apache.org Received: (qmail 91631 invoked by uid 500); 22 Nov 2013 20:47:46 -0000 Mailing-List: contact user-help@manifoldcf.apache.org; run by ezmlm Precedence: bulk List-Help: List-Unsubscribe: List-Post: List-Id: Reply-To: user@manifoldcf.apache.org Delivered-To: mailing list user@manifoldcf.apache.org Received: (qmail 91621 invoked by uid 99); 22 Nov 2013 20:47:46 -0000 Received: from nike.apache.org (HELO nike.apache.org) (192.87.106.230) by apache.org (qpsmtpd/0.29) with ESMTP; Fri, 22 Nov 2013 20:47:46 +0000 X-ASF-Spam-Status: No, hits=1.5 required=5.0 tests=HTML_MESSAGE,RCVD_IN_DNSWL_LOW,SPF_PASS X-Spam-Check-By: apache.org Received-SPF: pass (nike.apache.org: domain of mlibucha@gmail.com designates 209.85.128.170 as permitted sender) Received: from [209.85.128.170] (HELO mail-ve0-f170.google.com) (209.85.128.170) by apache.org (qpsmtpd/0.29) with ESMTP; Fri, 22 Nov 2013 20:47:39 +0000 Received: by mail-ve0-f170.google.com with SMTP id oy12so1306506veb.15 for ; Fri, 22 Nov 2013 12:47:18 -0800 (PST) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=gmail.com; s=20120113; h=mime-version:in-reply-to:references:date:message-id:subject:from:to :content-type; bh=VU7REeRZIFRvHXm1GlyRjgE5dEz7Ulba5l1zn7YciPo=; b=RYmyhNdeWIiFrqpY7F2kcjWte3Nz8eywSLgmOwnqsokMo3RkG3HIRQszHHoY+z81ct CDiIT8AhH9SaEgbwhDCfYG0jXijOf5/yGmzhShoYH4ocJdwn9kiZP+JbIXJZXZk1KG9h O5wLlKbBEFOylDGBE+en1kW8r8XwGwwo5BccIVxFHqNDhGXbubcPf5qd3s4lsehVZGVO Ox1J+3HCCouUadyak42zvzUeervqnQaSieR/jSJiteCLrzvscIu32W8ckuYwSCMp7M+A 56RQN2WrfQr/VQzJkTzKVzeKtNdjfc2ztSERKHpxizJVHw9GHp4aF3TUv9jJt9ATbMoM q/Rg== MIME-Version: 1.0 X-Received: by 10.58.255.71 with SMTP id ao7mr13139995ved.16.1385153238076; Fri, 22 Nov 2013 12:47:18 -0800 (PST) Received: by 10.59.3.201 with HTTP; Fri, 22 Nov 2013 12:47:17 -0800 (PST) In-Reply-To: References: Date: Fri, 22 Nov 2013 12:47:17 -0800 Message-ID: Subject: Re: SharePoint List item URLs From: Mark Libucha To: "user@manifoldcf.apache.org" Content-Type: multipart/alternative; boundary=047d7bf0d1b4b7177f04ebca1e04 X-Virus-Checked: Checked by ClamAV on apache.org --047d7bf0d1b4b7177f04ebca1e04 Content-Type: text/plain; charset=ISO-8859-1 I think you've done it! Patch installed without errors, and I'm getting the correct list item urls now. Unfortunately, a bug in my code was exposed so the crawl still isn't completing, but it's something I need to fix. Unless I update this thread again, it's all fixed. Many thanks, Karl! One final question: for this to work (list item urls), I need the patch AND the updated plugin, correct? Mark On Fri, Nov 22, 2013 at 12:28 PM, Karl Wright wrote: > Ok, done. > > - revert all previous changes > - apply the patch tagged with 1.4 > - this includes CONNECTORS-812.patch as well > > Karl > > > > On Fri, Nov 22, 2013 at 3:20 PM, Karl Wright wrote: > >> Didn't you also apply a patch to allow non-List items to not abort the >> crawl? That patch conflicts with this one, so I will try to include both. >> >> Karl >> >> >> On Fri, Nov 22, 2013 at 3:14 PM, Mark Libucha wrote: >> >>> Thanks! >>> >>> >>> On Fri, Nov 22, 2013 at 12:12 PM, Karl Wright wrote: >>> >>>> Ok, I'll need to redo the patch for release-1.4-branch it seems. Stay >>>> tuned. >>>> Karl >>>> >>>> >>>> On Fri, Nov 22, 2013 at 3:10 PM, Mark Libucha wrote: >>>> >>>>> How about this one? Also "fails": >>>>> >>>>> mlibucha@v4lnx3 apache-manifoldcf-1.4[master*]$ patch -p 0 < >>>>> ../patches/CONNECTORS-813-2.patch >>>>> >>>>> (Stripping trailing CRs from patch.) >>>>> patching file >>>>> connectors/sharepoint/connector/src/main/java/org/apache/manifoldcf/crawler/connectors/sharepoint/SPSProxyHelper.java >>>>> >>>>> (Stripping trailing CRs from patch.) >>>>> patching file >>>>> connectors/sharepoint/connector/src/main/java/org/apache/manifoldcf/crawler/connectors/sharepoint/SharePointRepository.java >>>>> Hunk #1 succeeded at 775 (offset -7 lines). >>>>> Hunk #2 succeeded at 803 (offset -7 lines). >>>>> Hunk #3 succeeded at 810 (offset -7 lines). >>>>> Hunk #4 succeeded at 831 (offset -7 lines). >>>>> Hunk #5 succeeded at 1372 (offset -7 lines). >>>>> Hunk #6 succeeded at 1439 (offset -7 lines). >>>>> Hunk #7 succeeded at 1490 (offset -7 lines). >>>>> Hunk #8 succeeded at 2177 (offset -7 lines). >>>>> Hunk #9 succeeded at 2216 (offset -7 lines). >>>>> Hunk #10 succeeded at 2246 (offset -7 lines). >>>>> Hunk #11 succeeded at 2256 (offset -7 lines). >>>>> Hunk #12 FAILED at 2286. >>>>> 1 out of 12 hunks FAILED -- saving rejects to file >>>>> connectors/sharepoint/connector/src/main/java/org/apache/manifoldcf/crawler/connectors/sharepoint/SharePointRepository.java.rej >>>>> >>>>> (Stripping trailing CRs from patch.) >>>>> patching file >>>>> connectors/sharepoint/connector/src/main/java/org/apache/manifoldcf/crawler/connectors/sharepoint/IFileStream.java >>>>> >>>>> >>>>> >>>>> On Fri, Nov 22, 2013 at 12:06 PM, Karl Wright wrote: >>>>> >>>>>> That's just CHANGES.txt. You can ignore that error. >>>>>> Karl >>>>>> >>>>>> >>>>>> On Fri, Nov 22, 2013 at 3:05 PM, Mark Libucha wrote: >>>>>> >>>>>>> When I most recent of the "original" 813 patch, it fails: >>>>>>> >>>>>>> (That it, this patch, dated Today 9:37, >>>>>>> https://issues.apache.org/jira/secure/attachment/12615359/CONNECTORS-813.patch >>>>>>> ) >>>>>>> >>>>>>> mlibucha@v4lnx3 apache-manifoldcf-1.4[master*]$ svn revert >>>>>>> connectors/sharepoint/connector/src/main/java/org/apache/manifoldcf/crawler/connectors/sharepoint/* >>>>>>> Reverted >>>>>>> 'connectors/sharepoint/connector/src/main/java/org/apache/manifoldcf/crawler/connectors/sharepoint/SharePointRepository.java' >>>>>>> Skipped >>>>>>> 'connectors/sharepoint/connector/src/main/java/org/apache/manifoldcf/crawler/connectors/sharepoint/SharePointRepository.java.orig' >>>>>>> mlibucha@v4lnx3 apache-manifoldcf-1.4[master*]$ patch -p 0 < >>>>>>> ../patches/CONNECTORS-813.patch >>>>>>> (Stripping trailing CRs from patch.) >>>>>>> patching file >>>>>>> connectors/sharepoint/connector/src/main/java/org/apache/manifoldcf/crawler/connectors/sharepoint/SharePointRepository.java >>>>>>> Hunk #1 succeeded at 796 (offset -7 lines). >>>>>>> Hunk #2 succeeded at 804 (offset -7 lines). >>>>>>> Hunk #3 succeeded at 826 (offset -7 lines). >>>>>>> Hunk #4 succeeded at 1366 (offset -7 lines). >>>>>>> Hunk #5 succeeded at 1485 (offset -7 lines). >>>>>>> (Stripping trailing CRs from patch.) >>>>>>> patching file CHANGES.txt >>>>>>> Hunk #1 FAILED at 3. >>>>>>> 1 out of 1 hunk FAILED -- saving rejects to file CHANGES.txt.rej >>>>>>> >>>>>>> >>>>>>> >>>>>>> On Fri, Nov 22, 2013 at 11:42 AM, Karl Wright wrote: >>>>>>> >>>>>>>> This was my mistake; I uploaded over the old patch. I since >>>>>>>> corrected that. Please just download the most recent of each. >>>>>>>> >>>>>>>> Karl >>>>>>>> >>>>>>>> >>>>>>>> On Fri, Nov 22, 2013 at 2:40 PM, Mark Libucha wrote: >>>>>>>> >>>>>>>>> >>>>>>>>> On Fri, Nov 22, 2013 at 11:27 AM, Karl Wright wrote: >>>>>>>>> >>>>>>>>>> CONNECTORS-813.patch >>>>>>>>>> >>>>>>>>> >>>>>>>>> >>>>>>>>> On last question...you mention "CONNECTORS-813.patch", but which >>>>>>>>> URL does this correspond to..there are 3 on that page: >>>>>>>>> >>>>>>>>> >>>>>>>>> https://issues.apache.org/jira/secure/attachment/12615359/CONNECTORS-813.patch >>>>>>>>> >>>>>>>>> https://issues.apache.org/jira/secure/attachment/12615357/CONNECTORS-813.patch >>>>>>>>> >>>>>>>>> https://issues.apache.org/jira/secure/attachment/12614943/CONNECTORS-813.patch >>>>>>>>> >>>>>>>>> >>>>>>>>> >>>>>>>> >>>>>>> >>>>>> >>>>> >>>> >>> >> > --047d7bf0d1b4b7177f04ebca1e04 Content-Type: text/html; charset=ISO-8859-1 Content-Transfer-Encoding: quoted-printable
I think you've done it!

Patch i= nstalled without errors, and I'm getting the correct list item urls now= .

Unfortunately, a bug in my code was exposed so the crawl sti= ll isn't completing, but it's something I need to fix. Unless I upd= ate this thread again, it's all fixed.

Many thanks, Karl!

One final question: for this to work (list it= em urls), I need the patch AND the updated plugin, correct?

Mark
=


On Fri,= Nov 22, 2013 at 12:28 PM, Karl Wright <daddywri@gmail.com>= wrote:
Ok, don= e.

- revert all previous changes
- apply the patch ta= gged with 1.4
- this includes CONNECTORS-812.patch as well

Karl



On Fri, Nov 22, 2013 at 3:20 PM, Karl Wr= ight <daddywri@gmail.com> wrote:
Didn't you also apply a patch to allow non-List items = to not abort the crawl?=A0 That patch conflicts with this one, so I will tr= y to include both.

Karl


On Fri, Nov 22, 2013 at 3:14 PM, Mark Libucha <mlibucha@gmail.com>= wrote:
Thanks!

=
On Fri, Nov 22, 2013 at 12:12 PM, Karl Wrigh= t <daddywri@gmail.com> wrote:
Ok, I'll need to redo t= he patch for release-1.4-branch it seems.=A0 Stay tuned.
Karl


On Fri, Nov 22, 2013 at 3:10 PM, Mark Libucha <m= libucha@gmail.com> wrote:
How about this one? Also &q= uot;fails":

mlibucha@v4lnx3 apache-manifoldcf-1.4[master*]$ pat= ch -p 0 < ../patches/CONNECTORS-813-2.patch

(Stripping trailing CRs from patch.)
patching file connectors/= sharepoint/connector/src/main/java/org/apache/manifoldcf/crawler/connectors= /sharepoint/SPSProxyHelper.java

(Stripping trailing CRs from patch.)
patching file connectors/sharepoint= /connector/src/main/java/org/apache/manifoldcf/crawler/connectors/sharepoin= t/SharePointRepository.java
Hunk #1 succeeded at 775 (offset -7 li= nes).
Hunk #2 succeeded at 803 (offset -7 lines).
Hunk #3 succeeded at 810 (of= fset -7 lines).
Hunk #4 succeeded at 831 (offset -7 lines).
Hunk #5 s= ucceeded at 1372 (offset -7 lines).
Hunk #6 succeeded at 1439 (offset -7= lines).
Hunk #7 succeeded at 1490 (offset -7 lines).
Hunk #8 succeeded at 2177 (= offset -7 lines).
Hunk #9 succeeded at 2216 (offset -7 lines).
Hunk #= 10 succeeded at 2246 (offset -7 lines).
Hunk #11 succeeded at 2256 (offs= et -7 lines).
Hunk #12 FAILED at 2286.
1 out of 12 hunks FAILED -- saving rejects to f= ile connectors/sharepoint/connector/src/main/java/org/apache/manifoldcf/cra= wler/connectors/sharepoint/SharePointRepository.java.rej

(Stripping trailing CRs from patch.)
patching file connectors/sharepoint/connector/src/main/java/org/apache/mani= foldcf/crawler/connectors/sharepoint/IFileStream.java



On Fri, Nov 22, 2013 at 12:06 PM, Karl Wright <daddywri@gmail.com>= wrote:
That's just CHANGE= S.txt.=A0 You can ignore that error.
Karl


= On Fri, Nov 22, 2013 at 3:05 PM, Mark Libucha <mlibucha@gmail.com>= wrote:
When I most recent of = the "original" 813 patch, it fails:

(That it, this = patch, dated Today 9:37, https://issues.a= pache.org/jira/secure/attachment/12615359/CONNECTORS-813.patch)

mlibucha@v4lnx3 apache-manifoldcf-1.4[master*]$ svn revert connect= ors/sharepoint/connector/src/main/java/org/apache/manifoldcf/crawler/connec= tors/sharepoint/*
Reverted 'connectors/sharepoint/connector/src/main= /java/org/apache/manifoldcf/crawler/connectors/sharepoint/SharePointReposit= ory.java'
Skipped 'connectors/sharepoint/connector/src/main/java/org/apache/manif= oldcf/crawler/connectors/sharepoint/SharePointRepository.java.orig'
= mlibucha@v4lnx3 apache-manifoldcf-1.4[master*]$ patch -p 0 < ../patches/= CONNECTORS-813.patch
(Stripping trailing CRs from patch.)
patching file connectors/sharepoint= /connector/src/main/java/org/apache/manifoldcf/crawler/connectors/sharepoin= t/SharePointRepository.java
Hunk #1 succeeded at 796 (offset -7 lines).<= br> Hunk #2 succeeded at 804 (offset -7 lines).
Hunk #3 succeeded at 826 (of= fset -7 lines).
Hunk #4 succeeded at 1366 (offset -7 lines).
Hunk #5 = succeeded at 1485 (offset -7 lines).
(Stripping trailing CRs from patch.= )
patching file CHANGES.txt
Hunk #1 FAILED at 3.
1 out of 1 hunk FAILED= -- saving rejects to file CHANGES.txt.rej



On Fri, Nov 22, 2013 at 11:42 AM, Karl Wright <daddywri@gmail.com>= wrote:
This was my mistake; I= uploaded over the old patch.=A0 I since corrected that.=A0 Please just dow= nload the most recent of each.

Karl


On Fri, Nov 22, 2013 at 2:40 PM, Mark Libucha <mlibucha@gmail.com>= wrote:

= On Fri, Nov 22, 2013 at 11:27 AM, Karl Wright <daddywri@gmail.com>= wrote:
CONNECTORS-813.patch=


On last q= uestion...you mention "CONNECTORS-813.patch", but which URL does = this correspond to..there are 3 on that page:

https://issues.apache.org/jira/secure/attachment/12615359/CONNE= CTORS-813.patch
https://issues.apache.org/jira/secure/att= achment/12615357/CONNECTORS-813.patch
https://issues.apache.org/jira/secure/attachment/12614943/CONNECTORS-81= 3.patch











--047d7bf0d1b4b7177f04ebca1e04--