Return-Path: X-Original-To: archive-asf-public-internal@cust-asf2.ponee.io Delivered-To: archive-asf-public-internal@cust-asf2.ponee.io Received: from cust-asf.ponee.io (cust-asf.ponee.io [163.172.22.183]) by cust-asf2.ponee.io (Postfix) with ESMTP id 32439200D13 for ; Sat, 16 Sep 2017 00:28:10 +0200 (CEST) Received: by cust-asf.ponee.io (Postfix) id 3119E1609D2; Fri, 15 Sep 2017 22:28:10 +0000 (UTC) Delivered-To: archive-asf-public@cust-asf.ponee.io Received: from mail.apache.org (hermes.apache.org [140.211.11.3]) by cust-asf.ponee.io (Postfix) with SMTP id CC4C11609D1 for ; Sat, 16 Sep 2017 00:28:08 +0200 (CEST) Received: (qmail 19046 invoked by uid 500); 15 Sep 2017 22:28:07 -0000 Mailing-List: contact dev-help@lucenenet.apache.org; run by ezmlm Precedence: bulk List-Help: List-Unsubscribe: List-Post: List-Id: Reply-To: dev@lucenenet.apache.org Delivered-To: mailing list dev@lucenenet.apache.org Received: (qmail 19034 invoked by uid 99); 15 Sep 2017 22:28:07 -0000 Received: from pnap-us-west-generic-nat.apache.org (HELO spamd3-us-west.apache.org) (209.188.14.142) by apache.org (qpsmtpd/0.29) with ESMTP; Fri, 15 Sep 2017 22:28:07 +0000 Received: from localhost (localhost [127.0.0.1]) by spamd3-us-west.apache.org (ASF Mail Server at spamd3-us-west.apache.org) with ESMTP id BF4E41865B4 for ; Fri, 15 Sep 2017 22:28:06 +0000 (UTC) X-Virus-Scanned: Debian amavisd-new at spamd3-us-west.apache.org X-Spam-Flag: NO X-Spam-Score: 6.165 X-Spam-Level: ****** X-Spam-Status: No, score=6.165 tagged_above=-999 required=6.31 tests=[DKIM_SIGNED=0.1, DKIM_VALID=-0.1, HTML_MESSAGE=2, HTTP_ESCAPED_HOST=1.621, KAM_LOTSOFHASH=0.25, KAM_NUMSUBJECT=0.5, RCVD_IN_DNSWL_NONE=-0.0001, RCVD_IN_MSPIKE_H3=-0.01, RCVD_IN_MSPIKE_WL=-0.01, RCVD_IN_SORBS_SPAM=0.5, URIBL_BLOCKED=0.001, URI_HEX=1.313] autolearn=disabled Authentication-Results: spamd3-us-west.apache.org (amavisd-new); dkim=pass (2048-bit key) header.d=darcythomas-com.20150623.gappssmtp.com Received: from mx1-lw-eu.apache.org ([10.40.0.8]) by localhost (spamd3-us-west.apache.org [10.40.0.10]) (amavisd-new, port 10024) with ESMTP id vMg3Z8cfVLlw for ; Fri, 15 Sep 2017 22:27:57 +0000 (UTC) Received: from mail-qk0-f169.google.com (mail-qk0-f169.google.com [209.85.220.169]) by mx1-lw-eu.apache.org (ASF Mail Server at mx1-lw-eu.apache.org) with ESMTPS id 521AD5FAF7 for ; Fri, 15 Sep 2017 22:27:56 +0000 (UTC) Received: by mail-qk0-f169.google.com with SMTP id a128so3311865qkc.5 for ; Fri, 15 Sep 2017 15:27:56 -0700 (PDT) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=darcythomas-com.20150623.gappssmtp.com; s=20150623; h=mime-version:references:in-reply-to:from:date:message-id:subject:to; bh=k/t+oYlAMP8Fk/MpbnbPfQsxUypKosNpqYn42omjrMo=; b=nNFrHmX7xM+xs9qQgoDHJrTRcCRjZyP2qUcuyYawYSEKgKg0/V2+Vxdbb2DHV7kyrG f02rRCJNov4Do7FI/Cqn2IMrqGj47eAlBrnuLM9m0BDbUe/uzvWOwmdf0dKAYlA14bYB qxmG7jG7ZnbB+ye7gF+IGRO8Oo1Cln39dY1heqWcFtI578YoTwcx/nohvpeJcKsctULC b3dZHow2bGNg08V5kIg81YIML1lB0LCLEe10uFZfo8SOtQiy6NlLU2V0/ZxwegiZl+cX rmFufbojBBZm+H/NhdylZUHfD+Dl/eipi8rgENhhBMXwsHjuyMiws8WT/zjwsQz07ezr CuIw== X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20161025; h=x-gm-message-state:mime-version:references:in-reply-to:from:date :message-id:subject:to; bh=k/t+oYlAMP8Fk/MpbnbPfQsxUypKosNpqYn42omjrMo=; b=fmF8ScHFQ/34Z3GcmleYjHtkTxqxVx9QT3xf4UHtT/Jg1guxOkSDc3KLZ/hFTvam0Q RSgy/7Omha8HINbxkeTh3Kyj00LVFUIL7meW4vFjUW8dzogfnFd6RDmZRoaa1xBii1yF RWMtAelP2rjy0fj3gPckQVSEemnlfBZvolW32Rg23EYPkVo96ToxRXpez+nSIyurAf65 Qqfhl71c4Grh0o2swv+fVVGSjQgO05MJinmPX4oOdGiBers6pexnayF/4zQBkuEtVt0a ndhz+I0UTYhgkF0i+mJ8CuAYA2tR7bn1cPJS6linNse3xoR3Ibt8m7+EiyzadOTQ3HkU 7klA== X-Gm-Message-State: AHPjjUhzN3qs+YC6AygywXU+gn3tKEE7dqkOFJAyukkv5gTZCuYG5ePj Buxqyg0rtFyTnAFRqDhmNpS1+okrCdp+Hx6z1AEAkg== X-Google-Smtp-Source: AOwi7QDaHIkMimwk71bexFUX+0BL1ENpqXgT9gx4CHi+q7LFpxrekxO6YmcYwDA+qLFAvkv2neRDbL4UAA8M8QSvVSg= X-Received: by 10.55.133.6 with SMTP id h6mr9683312qkd.17.1505514474846; Fri, 15 Sep 2017 15:27:54 -0700 (PDT) MIME-Version: 1.0 References: <458A3CD4F362D144B999930AAADEBAAD18E6EFA5@EX10DAG10-N1> <458A3CD4F362D144B999930AAADEBAAD18E6F03A@EX10DAG10-N1> In-Reply-To: From: Darcy Thomas Date: Fri, 15 Sep 2017 22:27:44 +0000 Message-ID: Subject: Re: Strong named assemblies for Lucene.Net 4.8 To: "dev@lucenenet.apache.org" Content-Type: multipart/alternative; boundary="94eb2c0766427a5d8d055941e766" archived-at: Fri, 15 Sep 2017 22:28:10 -0000 --94eb2c0766427a5d8d055941e766 Content-Type: text/plain; charset="UTF-8" Content-Transfer-Encoding: quoted-printable As a work around I have used https://www.nuget.org/packages/Brutal.Dev.StrongNameSigner/ on a past project to strongname (and fix up references) for any unsigned dependency. You just add the nuget package and it takes care of the rest. The let me GAC (ugh but it was the only way) everything I needed. On Sat, Sep 16, 2017, 10:07 Aaron Meyers wrote: > Thanks everyone for the replies, and sorry for my delay in responding. > > I have to agree that having two versions of a package seems like the wors= t > possible scenario and as a library, strong-naming doesn't seem to have an= y > real downsides as consumers can be strong-named or not. Strong naming > remains a requirement for my scenario (I did go back and ask but the > consensus was that the cost to remove across our large codebase is high a= nd > offers no real benefit to justify that cost) so I would love to see that > included in Lucene.Net 4.8. > > I'll follow up on what I need to do from our side to allow me to make > contributions to Lucene.Net. I'll need to get my index working with > Lucene.Net 4.8 and then I should be able to demonstrate where we are and > how to move forward for our release. > > We've had a lot of performance problems with VS2017 as well. I've > specifically worked with some people on the VS team about issues with > lightweight solution load but let me see if I can find some contacts for > more general performance issues and provide them the info you've given > below. > > Thanks again for all the help, > Aaron > > -----Original Message----- > From: Patric Forsgard [mailto:patric@tasteful.se] > Sent: Tuesday, September 12, 2017 10:44 PM > To: dev@lucenenet.apache.org > Subject: Re: Strong named assemblies for Lucene.Net 4.8 > > It exists some intresting comment threads on github where some of them > comes from users that during long time have say that strong naming is bad > and should bee avoided, but strong naming is less bad then having two > version that end's up with other problems. > > Comment ( > > https://na01.safelinks.protection.outlook.com/?url=3Dhttps%3A%2F%2Fgithub= .com%2Faspnet%2FDataProtection%2Fissues%2F245%23issuecomment-307155485&data= =3D02%7C01%7CAaron.Meyers%40microsoft.com%7C7b58a615c154486ce87908d4fa6a6e1= 5%7C72f988bf86f141af91ab2d7cd011db47%7C1%7C0%7C636408782411722200&sdata=3Dr= PnuZqmVdV4vdnTg6TkyThGgw43gLWA2DitoFI42eRQ%3D&reserved=3D0 > ) > is an answer about strong namer usage from David Fowler and says: > > ... that's for package consumers, not producers. Luckily, if you're > > using > .NET Core, there are no issues. The issue is .NET Framework and the only > saving grace is the automatic binding redirects that the build system > generates. Package authors realistically all need to be strong named and = we > (Microsoft) need to fix the .NET Framework to not require binding > redirects to truly solve the problems that people hit (I'm fighting for > this, again =F0=9F=98=84 > ). > > > Or this > > https://na01.safelinks.protection.outlook.com/?url=3Dhttps%3A%2F%2Fgithub= .com%2FAutoMapper%2FAutoMapper.Collection%2Fpull%2F43%23issuecomment-312351= 176&data=3D02%7C01%7CAaron.Meyers%40microsoft.com%7C7b58a615c154486ce87908d= 4fa6a6e15%7C72f988bf86f141af91ab2d7cd011db47%7C1%7C0%7C636408782411722200&s= data=3DhytV5uDqDLvCXtuSbc%2FjTBXbDXAPXpkUj55cYVn5uUs%3D&reserved=3D0 > that also have the conclusion from Jimmy Bogard that it is better to only > have one package instead of multiple package that. > > > // Patric > > On 13 September 2017 at 06:57, Daniel Plaisted < > daplaist@microsoft.com.invalid> wrote: > > > Hi Aaron, > > > > Consider using Strong Namer: > > https://na01.safelinks.protection.outlook.com/?url=3Dhttps%3A%2F%2Fgith= u > > b.com%2Fdsplaisted%2Fstrongnamer&data=3D02%7C01%7CAaron.Meyers%40micros= o > > ft.com%7C7b58a615c154486ce87908d4fa6a6e15%7C72f988bf86f141af91ab2d7cd0 > > 11db47%7C1%7C0%7C636408782411722200&sdata=3D64n7kbEA1laUvyUV2i1Y4ep5v%2= B > > Is7xs5LODuENhoD88%3D&reserved=3D0 > > > > This is a project of mine that signs referenced assemblies as part of > > the build process, so you can reference libraries that aren't signed > > from projects that are. > > > > Thanks, > > Daniel > > > > -----Original Message----- > > From: Shad Storhaug [mailto:shad@shadstorhaug.com] > > Sent: Tuesday, September 12, 2017 9:28 PM > > To: Aaron Meyers > > Cc: dev@lucenenet.apache.org > > Subject: RE: Strong named assemblies for Lucene.Net 4.8 > > > > Aaron, > > > > BTW - if it were only up to me I would just strong name the assemblies > > and not worry about it. But this is something we should put to a vote > > since some of the team members have actively spoken out against it. > > > > If you were to commit to doing some contributions (for example, > > finishing up the ICU functionality and/or contributing a couple of UI > > integration modules described at > > https://na01.safelinks.protection.outlook.com/?url=3D > > https%3A%2F%2Fgithub.com%2Fapache%2Flucenenet%2Fpull%2F209%23issuecomm > > ent- 321019002&data=3D02%7C01%7Cdaplaist%40microsoft.com% > > 7C580735c68cf14a53cbe408d4fa5fd0ca%7C72f988bf86f141af91ab2d7cd011 > > db47%7C1%7C0%7C636408736822148766&sdata=3DflixkESIVLPuCgmRD4HkYBiD% > > 2BxJyX0B73gNWJ7sMwhc%3D&reserved=3D0), it might be an alternative way t= o > > get this done rather than opening up the issue on JIRA. I think that > > would be enough to convince the team that this is the best choice for > > all of our interests, and would be a more realistic avenue to get this > > done for the next beta release. Since we have very limited resources > > at our disposal, it would be difficult for anyone on the team to say no > to an offer like that. > > > > Thanks, > > Shad Storhaug (NightOwl888) > > > > > > -----Original Message----- > > From: Shad Storhaug [mailto:shad@shadstorhaug.com] > > Sent: Wednesday, September 13, 2017 5:24 AM > > To: Aaron.Meyers@microsoft.com; synhershko; > > itamar.synhershko@gmail.com > > Cc: dev@lucenenet.apache.org > > Subject: RE: Strong named assemblies for Lucene.Net 4.8 > > > > Aaron, > > > > Thanks for your interest in Lucene.Net. Answers inline... > > > > > Are there any plans to strong-name the Lucene.Net 4.8 assemblies in > > > the > > beta Nuget packages? I'm assuming that when the strong name is added > > to the Lucene.Net assembly it will use the Lucene.Net.snk already > > present in the repo so we should go ahead and use that key as well? > > > > I have no objections and understand this is a serious blocker for > > anyone who requires strong naming, however since you are literally > > only the second person to ask for this I repeat what I told the last > > person. Please, open an issue on JIRA about strong naming and > > encourage people to vote on it if it is a blocker for them by sending > > an email to our user mailing list (or provide any feedback why we > > shouldn't do it). Including information about your expected user base > would also be something for us to consider. > > "Because it is a blocker if someone requires it" isn't quite enough to > > make a case for a feature that we are not sure anyone uses anymore, > > but I'd be happy to advocate for it if we have more evidence this is > > still needed by some people. > > > > Do note that all of our dependencies are strong named AFAIK and I have > > setup the versioning with strong naming in mind, so this shouldn't > > take a huge effort to do. > > > > If you are willing to help we would accept a contribution to enable > > strong naming *as an option* during build (FEATURE_STRONG_NAME), which > > is something that is on my TODO list. And yes, we will be using the > > Lucene.Net.snk in our repository for this. > > > > > Is there some specific investigation needed that I could do? > > > > See above. > > > > > I'm curious though why 4.5.1 was targeted specifically for > > > Lucene.Net > > since there weren't many significant changes in that version? > > > > This is a question that I have also asked, but the person/people who > > made that decision are no longer actively involved in the project. > > > > > Does anyone have more context on the problem? I don't recall any > > > issues > > with unmanaged references and strong names (we have both unmanaged > > references from C# DLLs and Managed C++ DLLs in our codebase) but > > might not fully understand the comment. I could look into this further > > but wondering if anyone has more information before I try to start blin= d. > > > > This comment is now out of date (actually, the first sentence has been > > removed from the codebase since you looked). Basically, we were unable > > to strong name Lucene.Net.Analysis.Common previously because of a > > reference to ICU4NET, which has since been changed to icu.net. ICU4NET > > was a blocker for the SNK, but now it is no longer an issue. The > > dependency has also been consolidated into a new project named > > Lucene.Net.ICU to prevent the majority of our users from having a 25 > > MB dependency that a small minority will actually need. > > > > > Any timeline for the next beta release? I've seen mention recently > > > that > > another beta release is planned soon. Any timeline on this? > > > > "Soon" is a relative term, which of course depends on how quickly we > > can get through all of the tooling hurdles and bug fixes and how much > > help we have to do it. I just sent an email to our dev list about some > > of the issues we are facing with failing tests ( > https://na01.safelinks.protection.outlook.com/?url=3Dhttps%3A%2F%2Fna01.s= afelinks&data=3D02%7C01%7CAaron.Meyers%40microsoft.com%7C7b58a615c154486ce8= 7908d4fa6a6e15%7C72f988bf86f141af91ab2d7cd011db47%7C1%7C0%7C636408782411732= 208&sdata=3D%2FlUTbzRBJk3CMQcpAJG4iMKAznEWdq6ObjBDESH6F0g%3D&reserved=3D0 > . > > protection.outlook.com/?url=3Dhttp%3A%2F%2Fapache.markmail.org%2Fmessag= e > > % > > 2F3kgnc7bguo73kmbw%3Fq%3Dlucenenet%2Bdebugging%2Bhelp%2Brequested%2Bfo > > r% 2Bbeta&data=3D02%7C01%7Cdaplaist%40microsoft.com% > > 7C580735c68cf14a53cbe408d4fa5fd0ca%7C72f988bf86f141af91ab2d7cd011 > > db47%7C1%7C0%7C636408736822148766&sdata=3D7r46aivOu45xrq%2FMH5RHKnIUQi% > > 2F5V2aElk4Ayg0n44k%3D&reserved=3D0), and any help you could provide > > would also be welcome. > > > > Basically, there are 4 things I would like to include in the release > > that are not yet done: > > > > 1. We are currently getting random "file in use by another process" > > exceptions in the tests, which I don't believe were happening in the > > last release. Need to confirm that these issues were not present in > > the last release, find out which commit caused them to start, and decid= e > on a fix. > > 2. There is a new .NET Core 2.0 command line tool (lucene-cli) that > > needs to be published and packaged in the build (ideally on > > Chocolatey, but just zipped and included in the Lucene.Net distribution > https://na01.safelinks.protection.outlook.com/?url=3Dhttps%3A%2F%2Fna01.s= afelinks&data=3D02%7C01%7CAaron.Meyers%40microsoft.com%7C7b58a615c154486ce8= 7908d4fa6a6e15%7C72f988bf86f141af91ab2d7cd011db47%7C1%7C0%7C636408782411732= 208&sdata=3D%2FlUTbzRBJk3CMQcpAJG4iMKAznEWdq6ObjBDESH6F0g%3D&reserved=3D0 > . > > protection.outlook.com/?url=3Dhttp%3A%2F%2Fwww.apache.org% > > 2Fdist%2Flucenenet%2F&data=3D02%7C01%7Cdaplaist%40microsoft.com% > > 7C580735c68cf14a53cbe408d4fa5fd0ca%7C72f988bf86f141af91ab2d7cd011 > > db47%7C1%7C0%7C636408736822148766&sdata=3DLiZiyhv1HMcuz2Ue%2B% > > 2B6tUKmoJOzXWEsqlBgTmyPAhU0%3D&reserved=3D0 failing that). > > 3. Include API documentation generation as part of the build. See > > https://na01.safelinks.protection.outlook.com/?url=3D > > https%3A%2F%2Fgithub.com%2Fapache%2Flucenenet%2Fpull% > > 2F206&data=3D02%7C01%7Cdaplaist%40microsoft.com% > > 7C580735c68cf14a53cbe408d4fa5fd0ca%7C72f988bf86f141af91ab2d7cd011 > > db47%7C1%7C0%7C636408736822148766&sdata=3Dhfk1qGDYP1HiZMl9bKnBYqhUOqNKM= M > > KhVRUymbEOGds%3D&reserved=3D0. > > 4. Change from .NET Framework 4.5.1 to .NET Framework 4.5. This was > > also requested previously and according to the requestor the 3 > > assemblies that would most likely be problematic all compiled fine. > > > > Ideally, we would also like to get all of the tests passing, too, but > > I am not going to hold up the release if the ICU issues are not fixed. > > > > If you know anything about packaging a .NET Core .DLL command line > > utility as a Chocolatey package, we could use the help with that as > > well. It doesn't seem to be documented anywhere how to make a package > > for a .NET Core .DLL (Chocolatey only considers .EXE files by default). > > > > And if there is any strings you can pull at Microsoft to make the > > tooling run faster, that would also help :). Currently it takes: > > > > 1. ~10 minutes to discover the tests in Visual Studio 2017 (in VS 2015 > > this was a minute or two) 2. ~20 minutes to install the > > SDK/restore/build/pack on the command line (with the project.json and > > preview tooling, this was about 5 minutes) 3. Since there is no way to > > switch target framework in VSTest, we are doing this manually in a > > linked MSBuild file. But Visual Studio usually needs to be closed and > > reopened again when it is changed, which means we are at least 10 > > minutes away from being able to run the tests whenever it is switched. > > Previously, we had 2 different solution files. Now it is looking like > > the most practical thing to do will be to make a solution file per > assembly/test assembly pair. > > > > > Additionally, I came across this blog post< > https://na01.safelinks.protection.outlook.com/?url=3Dhttps%3A%2F%2Fna01.s= afelinks&data=3D02%7C01%7CAaron.Meyers%40microsoft.com%7C7b58a615c154486ce8= 7908d4fa6a6e15%7C72f988bf86f141af91ab2d7cd011db47%7C1%7C0%7C636408782411732= 208&sdata=3D%2FlUTbzRBJk3CMQcpAJG4iMKAznEWdq6ObjBDESH6F0g%3D&reserved=3D0 > . > > protection.outlook.com/?url=3Dhttp%3A%2F%2Fcode972.com% > > 2Fblog%2F2016%2F07%2F98-lucene-net-4-8-is-in-beta-and- > > we-need-your-help&data=3D02%7C01%7Cdaplaist%40microsoft.com% > > 7C580735c68cf14a53cbe408d4fa5fd0ca%7C72f988bf86f141af91ab2d7cd011 > > db47%7C1%7C0%7C636408736822148766&sdata=3DN%2BCfIu59XAFix1Av8lyjNsDZ24% > > 2BkT48ozn8bwwnhpus%3D&reserved=3D0> which suggests that Lucene.Net 4.8 > > should be a good option even in its current "beta" form. > > > > We are now quite a bit better off from when that blog post was > > written. I cannot comment on whether we are more/less stable than > > Lucene 3.0.3, but our goal is to get it as stable as Java Lucene > > before making the release official, and then work on closing the > > version gap. We have a few tests that are newly failing after > > upgrading from project.json to .csproj, but considering that we have > > nearly 8000 tests that pass I'd say we are on track to meet that goal. > > > > > The blog post referenced above also mentioned that a significant > > > amount > > of contribution to Lucene.Net 4.8 was done by Microsoft employees. > > Does anyone have names or contact info for some of these people? I > > would love to reach out internally to understand these contributions > > and how Lucene.Net > > 4.8 is being used elsewhere at Microsoft as this will help us > > coordinate our plans and any potential contributions. > > > > The only Microsoft employee that I have crossed paths with during the > > year or so I have been involved with the project is Connie Yau, who > > primarily helped us get on .NET Standard and helped out a lot with the > > International Components for Unicode (ICU) part, which still has a bit > > more work left to complete. See: > > > > 1. https://na01.safelinks.protection.outlook.com/?url=3D > > https%3A%2F%2Fgithub.com%2Fsillsdev%2Ficu-dotnet%2Fpull%2F37&data=3D02%= 7 > > C01% 7Cdaplaist%40microsoft.com%7C580735c68cf14a53cbe408d4fa5fd0ca% > > 7C72f988bf86f141af91ab2d7cd011db47%7C1%7C0%7C636408736822148766&sdata= =3D > > kjTZrwxtB8Yh%2FmdHxcwTuIGzgKeGzvhP3Kbwhr%2BrlEc%3D&reserved=3D0 > > 2. https://na01.safelinks.protection.outlook.com/?url=3D > > https%3A%2F%2Fgithub.com%2Fapache%2Flucenenet%2Ftree% > > 2Fmaster%2Fsrc%2FLucene.Net.Analysis.ICU%2FAnalysis%2FIcu& > > data=3D02%7C01%7Cdaplaist%40microsoft.com%7C580735c68cf14a53cbe408d4fa5= f > > d0ca%7C72f988bf86f141af91ab2d7cd011db47%7C1%7C0% > > 7C636408736822148766&sdata=3DKl%2BgymY%2BW709PbCv2Pi%2B2% > > 2FBA56eAqkdT%2Bk878oXMzeI%3D&reserved=3D0 > > > > > > @Itamar, do you know of any others? > > > > > > Thanks, > > Shad Storhaug (NightOwl888) > > > > > > -----Original Message----- > > From: itamar.synhershko@gmail.com [mailto:itamar.synhershko@gmail.com] > > On Behalf Of Itamar Syn-Hershko > > Sent: Tuesday, September 12, 2017 2:37 AM > > To: dev@lucenenet.apache.org > > Subject: Re: Strong named assemblies for Lucene.Net 4.8 > > > > Hi Aaron, > > > > With regards to strong naming, please see this: > > https://na01.safelinks.protection.outlook.com/?url=3D > > http%3A%2F%2Fcode972.com%2Fblog%2F2014%2F04%2F68- > > ditching-strong-naming-for-lucene-net&data=3D02%7C01% > > 7Cdaplaist%40microsoft.com%7C580735c68cf14a53cbe408d4fa5fd0ca% > > 7C72f988bf86f141af91ab2d7cd011db47%7C1%7C0%7C636408736822148766&sdata= =3D > > IIshOtISyeW66VGxZmsKCOz%2FoZ94rv8GX4eChWIVWXs%3D&reserved=3D0 > > > > As Lucene.NET is an Apache project, all decisions have to go through > > the community and developers so I can't really make a decision here. I > > do hope the reasoning above would make sense to all involved. You can > > sign the assemblies yourself if needed. > > > > Cheers, > > > > -- > > > > Itamar Syn-Hershko > > Freelance Developer & Consultant > > Elasticsearch Partner > > Microsoft MVP | Lucene.NET PMC > > https://na01.safelinks.protection.outlook.com/?url=3D > > http%3A%2F%2Fcode972.com&data=3D02%7C01%7Cdaplaist%40microsoft.com% > > 7C580735c68cf14a53cbe408d4fa5fd0ca%7C72f988bf86f141af91ab2d7cd011 > > db47%7C1%7C0%7C636408736822148766&sdata=3DaU1F%2BWnorMjjPtIDv3Lju1jTU% > > 2BLqOZ8xSJ1cHlacl%2FU%3D&reserved=3D0 | @synhershko < > https://na01.safelinks.protection.outlook.com/?url=3Dhttps%3A%2F%2Fna01.s= afelinks&data=3D02%7C01%7CAaron.Meyers%40microsoft.com%7C7b58a615c154486ce8= 7908d4fa6a6e15%7C72f988bf86f141af91ab2d7cd011db47%7C1%7C0%7C636408782411732= 208&sdata=3D%2FlUTbzRBJk3CMQcpAJG4iMKAznEWdq6ObjBDESH6F0g%3D&reserved=3D0 > . > > protection.outlook.com/?url=3Dhttps%3A%2F%2Ftwitter.com% > > 2Fsynhershko&data=3D02%7C01%7Cdaplaist%40microsoft.com% > > 7C580735c68cf14a53cbe408d4fa5fd0ca%7C72f988bf86f141af91ab2d7cd011 > > db47%7C1%7C0%7C636408736822148766&sdata=3DTXxKsDztnL0qABRSVyzPgygSFb4zq= G > > nyWatozGiwsoc%3D&reserved=3D0> > https://na01.safelinks.protection.outlook.com/?url=3Dhttps%3A%2F%2Fna01.s= afelinks&data=3D02%7C01%7CAaron.Meyers%40microsoft.com%7C7b58a615c154486ce8= 7908d4fa6a6e15%7C72f988bf86f141af91ab2d7cd011db47%7C1%7C0%7C636408782411732= 208&sdata=3D%2FlUTbzRBJk3CMQcpAJG4iMKAznEWdq6ObjBDESH6F0g%3D&reserved=3D0 > . > > protection.outlook.com/?url=3Dhttp%3A%2F%2FBigDataBoutique. > > co.il%2F&data=3D02%7C01%7Cdaplaist%40microsoft.com% > > 7C580735c68cf14a53cbe408d4fa5fd0ca%7C72f988bf86f141af91ab2d7cd011 > > db47%7C1%7C0%7C636408736822148766&sdata=3D%2FuFc7HnBUdFzwDF39JiMlPJPvXi= 4 > > p > > oUcgMIFYqOxqjs%3D&reserved=3D0 > > > > On Mon, Sep 11, 2017 at 10:33 PM, Aaron Meyers < > > Aaron.Meyers@microsoft.com.invalid> wrote: > > > > > tl;dr Are there any plans to strong-name the Lucene.Net 4.8 > > > assemblies in the beta Nuget packages? Is there some specific > > > investigation needed that I could do? Any timeline for the next beta > > > release? And does anyone know why > > > 4.5.1 was chosen as the target framework for Lucene.Net 4.8? > > > > > > Quick intro: I'm an engineering lead for the language understanding > > > engine underlying the "Q&A" natural language query feature in > > > Microsoft's Power BI product (internally known as "Lucia"). We would > > > like to start using Lucene.Net to implement our index of the user's > > > database string values (replacing current internal solutions which > > > have some drawbacks). Our scenario is rather different from the > > > typical application of Lucene but still needs much of the same > > underlying algorithms/features. > > > > > > I have a working implementation over Lucene.Net 3.0.3 but recently > > > discovered that Lucene.Net 4.8 supports both store compression and > > > memory-mapped files which would be useful for our scenario. > > > Additionally, I came across this blog > > > post > > Fc > > > ode972.com%2Fblog%2F&data=3D02%7C01%7Cdaplaist%40microsoft.com%7C5807= 3 > > > 5c > > > 68cf14a53cbe408d4fa5fd0ca%7C72f988bf86f141af91ab2d7cd011db47%7C1%7C0 > > > %7 > > > C636408736822148766&sdata=3DnbyJVm3UNeodmDrq93rQviAZi8gGeBCJ75RelKCpH= o > > > E% > > > 3D&reserved=3D0 > > > 2016/07/98-lucene-net-4-8-is-in-beta-and-we-need-your-help> which > > > suggests that Lucene.Net 4.8 should be a good option even in its > > > current "beta" form. > > > > > > As I've started working with Lucene.Net 4.8, I hit two early > roadblocks: > > > > > > 1. All of our assemblies are strong-named but the current > > > Lucene.Net > > > 4.8 beta packages on Nuget are not strong-named > > > 2. Our codebase currently targets .NET 4.5 but Lucene.Net 4.8 > > > targets > > > 4.5.1 > > > > > > For 1), I was able to build a strong-named version of Lucene.Net 4.8 > > > by cloning the git repo and using the Lucene.Net.snk included in the > > > repo. I had to comment out these lines in the AssemblyInfo.cs for > > > Lucene.Net (I haven't tried to build any other assemblies yet - for > > > our usage, we'll probably only need the core Lucene.Net.dll as we'll > > > provide our own analyzers and we build queries directly): > > > // LUCENENET NOTE: For now it is not possible to use a SNK because > > > we have unmanaged references in Analysis.Common. > > > // However, we still need InternalsVisibleTo in order to prevent > > > making everything public just for the sake of testing. > > > // This has broad implications, though because many methods are > > > marked "protected internal", which means other assemblies // must > > > update overridden methods to match. > > > /* > > > [assembly: InternalsVisibleTo("Lucene.Net.Tests")] > > > [assembly: InternalsVisibleTo("Lucene.Net.TestFramework")] > > > [assembly: InternalsVisibleTo("Lucene.Net.Highlighter")] // For > > > Automaton > > > [assembly: InternalsVisibleTo("Lucene.Net.ICU")] // For Automaton > > > [assembly: InternalsVisibleTo("Lucene.Net.Misc")] > > > [assembly: InternalsVisibleTo("Lucene.Net.Suggest")] // For > > > Automaton > > > [assembly: InternalsVisibleTo("Lucene.Net.Tests.Analysis.Common")] > > > // For Automaton > > > [assembly: InternalsVisibleTo("Lucene.Net.Tests.Highlighter")] // > > > For Automaton > > > [assembly: InternalsVisibleTo("Lucene.Net.Tests.ICU")] // For > > > Analysis.Util.TestSegmentingTokenizerBase > > > [assembly: InternalsVisibleTo("Lucene.Net.Tests.Misc")] > > > [assembly: InternalsVisibleTo("Lucene.Net.Tests.QueryParser")] > > > [assembly: InternalsVisibleTo("Lucene.Net.Tests.Cli")] // For > > > lucene-cli */ > > > > > > I didn't see any issue in JIRA for this comment. Does anyone have > > > more context on the problem? I don't recall any issues with > > > unmanaged references and strong names (we have both unmanaged > > > references from C# DLLs and Managed C++ DLLs in our codebase) but > > > might not fully understand the comment. I could look into this > > > further but wondering if anyone has more information before I try to > start blind. > > > > > > For 2), I also changed the target framework in the assembly I built > > > to temporarily unblock our development. We are planning to move to > > > 4.5.2 target since both 4.5 and 4.5.1 haven't actually been > > > supported since > > 2016. > > > I'm curious though why 4.5.1 was targeted specifically for > > > Lucene.Net since there weren't many significant changes in that > > > version? In any case though this shouldn't be a blocker for us > > > (while we'll do some development on the > > > 4.5 target at the moment, we're of course running 4.6 or later on > > > our development machines so I don't expect any differences between > > > 4.5 and > > > 4.5.1 target). > > > > > > Two other questions: > > > > > > 1. I've seen mention recently that another beta release is > > > planned soon. Any timeline on this? We can ship a strong-named > > > version of Lucene.Net built ourselves but would prefer an official > > > release if possible. I'm assuming that when the strong name is added > > > to the Lucene.Net assembly it will use the Lucene.Net.snk already > > > present in the repo so we should go ahead and use that key as well? > > > 2. The blog post referenced above also mentioned that a > > > significant amount of contribution to Lucene.Net 4.8 was done by > Microsoft employees. > > > Does anyone have names or contact info for some of these people? I > > > would love to reach out internally to understand these contributions > > > and how Lucene.Net 4.8 is being used elsewhere at Microsoft as this > > > will help us coordinate our plans and any potential contributions. > > > > > > Thanks for your help and all the great work on Lucene.Net 4.8. We're > > > very excited to be able to leverage this in our product. > > > > > > Aaron Meyers > > > Senior Software Engineer > > > Microsoft Power BI > > > > > > > > > --94eb2c0766427a5d8d055941e766--