NetBSD Problem Report #51485

From www@NetBSD.org  Sun Sep 18 10:41:28 2016
Return-Path: <www@NetBSD.org>
Received: from mail.netbsd.org (mail.netbsd.org [199.233.217.200])
	(using TLSv1.2 with cipher ECDHE-RSA-AES256-GCM-SHA384 (256/256 bits))
	(Client CN "mail.netbsd.org", Issuer "Postmaster NetBSD.org" (verified OK))
	by mollari.NetBSD.org (Postfix) with ESMTPS id DAAF17A210
	for <gnats-bugs@gnats.NetBSD.org>; Sun, 18 Sep 2016 10:41:27 +0000 (UTC)
Message-Id: <20160918104126.E207C7A2BC@mollari.NetBSD.org>
Date: Sun, 18 Sep 2016 10:41:26 +0000 (UTC)
From: js-pkgsrc@heap.zone
Reply-To: js-pkgsrc@heap.zone
To: gnats-bugs@NetBSD.org
Subject: pkgsrc cannot be checked out on a case-insensitive file system
X-Send-Pr-Version: www-1.0

>Number:         51485
>Category:       pkg
>Synopsis:       pkgsrc cannot be checked out on a case-insensitive file system
>Confidential:   no
>Severity:       serious
>Priority:       medium
>Responsible:    joerg
>State:          closed
>Class:          sw-bug
>Submitter-Id:   net
>Arrival-Date:   Sun Sep 18 10:45:00 +0000 2016
>Closed-Date:    Sun Nov 20 12:30:51 +0000 2016
>Last-Modified:  Sun Nov 20 12:30:51 +0000 2016
>Originator:     Jonathan Schleifer
>Release:        HEAD
>Organization:
>Environment:
Darwin obuolys.local 16.0.0 Darwin Kernel Version 16.0.0: Mon Aug 29 17:56:20 PDT 2016; root:xnu-3789.1.32~3/RELEASE_X86_64 x86_64
>Description:
The following two files are currently in pkgsrc:

www/seamonkey/patches/patch-mozilla_gfx_ycbcr_yuv__row__arm.s
www/seamonkey/patches/patch-mozilla_gfx_ycbcr_yuv__row__arm.S

This creates a conflict on case-insensitive file systems, where one overwrites the other. This leads to git refusing certain operations, as it constantly thinks there are changes, even though there aren't (e.g. a rebase is impossible).
>How-To-Repeat:
git clone https://github.com/jsonn/pkgsrc (or use cvs) on a case-insensitive file system
>Fix:
Rename one of the two files

>Release-Note:

>Audit-Trail:

State-Changed-From-To: open->closed
State-Changed-By: wiz@NetBSD.org
State-Changed-When: Sun, 18 Sep 2016 11:15:50 +0000
State-Changed-Why:
Fixed, thanks.


From: "Thomas Klausner" <wiz@netbsd.org>
To: gnats-bugs@gnats.NetBSD.org
Cc: 
Subject: PR/51485 CVS commit: pkgsrc/www/seamonkey
Date: Sun, 18 Sep 2016 11:09:42 +0000

 Module Name:	pkgsrc
 Committed By:	wiz
 Date:		Sun Sep 18 11:09:42 UTC 2016

 Modified Files:
 	pkgsrc/www/seamonkey: distinfo
 Removed Files:
 	pkgsrc/www/seamonkey/patches: patch-mozilla_gfx_ycbcr_yuv__row__arm.S

 Log Message:
 Remove patch that adds a .s file as .S file.

 I can't believe both copies are needed.

 Addresses "pkgsrc cannot be checked out on a case-insensitive file system"
 by Jonathan Schleifer, PR 51485.


 To generate a diff of this commit:
 cvs rdiff -u -r1.136 -r1.137 pkgsrc/www/seamonkey/distinfo
 cvs rdiff -u -r1.1 -r0 \
     pkgsrc/www/seamonkey/patches/patch-mozilla_gfx_ycbcr_yuv__row__arm.S

 Please note that diffs are not public domain; they are subject to the
 copyright notices on the relevant files.

From: Joerg Sonnenberger <joerg@bec.de>
To: gnats-bugs@NetBSD.org
Cc: pkg-manager@netbsd.org, gnats-admin@netbsd.org, pkgsrc-bugs@netbsd.org,
	js-pkgsrc@heap.zone
Subject: Re: PR/51485 CVS commit: pkgsrc/www/seamonkey
Date: Mon, 19 Sep 2016 17:00:54 +0200

 On Sun, Sep 18, 2016 at 11:20:00AM +0000, Thomas Klausner wrote:
 > The following reply was made to PR pkg/51485; it has been noted by GNATS.
 > 
 > From: "Thomas Klausner" <wiz@netbsd.org>
 > To: gnats-bugs@gnats.NetBSD.org
 > Cc: 
 > Subject: PR/51485 CVS commit: pkgsrc/www/seamonkey
 > Date: Sun, 18 Sep 2016 11:09:42 +0000
 > 
 >  Module Name:	pkgsrc
 >  Committed By:	wiz
 >  Date:		Sun Sep 18 11:09:42 UTC 2016
 >  
 >  Modified Files:
 >  	pkgsrc/www/seamonkey: distinfo
 >  Removed Files:
 >  	pkgsrc/www/seamonkey/patches: patch-mozilla_gfx_ycbcr_yuv__row__arm.S
 >  
 >  Log Message:
 >  Remove patch that adds a .s file as .S file.
 >  
 >  I can't believe both copies are needed.
 >  
 >  Addresses "pkgsrc cannot be checked out on a case-insensitive file system"
 >  by Jonathan Schleifer, PR 51485.

 This is wrong. The .s version doesn't work as is because it won't get
 preprocessed first.

 Joerg

From: Thomas Klausner <wiz@NetBSD.org>
To: NetBSD bugtracking <gnats-bugs@NetBSD.org>,
	Joerg Sonnenberger <joerg@britannica.bec.de>
Cc: 
Subject: Re: PR/51485 CVS commit: pkgsrc/www/seamonkey
Date: Mon, 19 Sep 2016 17:26:34 +0200

 On Mon, Sep 19, 2016 at 05:00:54PM +0200, Joerg Sonnenberger wrote:
 > On Sun, Sep 18, 2016 at 11:20:00AM +0000, Thomas Klausner wrote:
 > > The following reply was made to PR pkg/51485; it has been noted by GNATS.
 > > 
 > > From: "Thomas Klausner" <wiz@netbsd.org>
 > > To: gnats-bugs@gnats.NetBSD.org
 > > Cc: 
 > > Subject: PR/51485 CVS commit: pkgsrc/www/seamonkey
 > > Date: Sun, 18 Sep 2016 11:09:42 +0000
 > > 
 > >  Module Name:	pkgsrc
 > >  Committed By:	wiz
 > >  Date:		Sun Sep 18 11:09:42 UTC 2016
 > >  
 > >  Modified Files:
 > >  	pkgsrc/www/seamonkey: distinfo
 > >  Removed Files:
 > >  	pkgsrc/www/seamonkey/patches: patch-mozilla_gfx_ycbcr_yuv__row__arm.S
 > >  
 > >  Log Message:
 > >  Remove patch that adds a .s file as .S file.
 > >  
 > >  I can't believe both copies are needed.
 > >  
 > >  Addresses "pkgsrc cannot be checked out on a case-insensitive file system"
 > >  by Jonathan Schleifer, PR 51485.
 > 
 > This is wrong. The .s version doesn't work as is because it won't get
 > preprocessed first.

 So we only need the .S patch and can remove the .s one?
  Thomas

From: Thomas Klausner <wiz@NetBSD.org>
To: Joerg Sonnenberger <joerg@bec.de>
Cc: NetBSD bugtracking <gnats-bugs@NetBSD.org>
Subject: Re: PR/51485 CVS commit: pkgsrc/www/seamonkey
Date: Mon, 19 Sep 2016 22:37:01 +0200

 On Mon, Sep 19, 2016 at 10:17:49PM +0200, Joerg Sonnenberger wrote:
 > On Mon, Sep 19, 2016 at 10:05:48PM +0200, Thomas Klausner wrote:
 > > On Mon, Sep 19, 2016 at 06:10:12PM +0200, Joerg Sonnenberger wrote:
 > > > On Mon, Sep 19, 2016 at 05:26:34PM +0200, Thomas Klausner wrote:
 > > > > On Mon, Sep 19, 2016 at 05:00:54PM +0200, Joerg Sonnenberger wrote:
 > > > > > On Sun, Sep 18, 2016 at 11:20:00AM +0000, Thomas Klausner wrote:
 > > > > > > The following reply was made to PR pkg/51485; it has been noted by GNATS.
 > > > > > > 
 > > > > > > From: "Thomas Klausner" <wiz@netbsd.org>
 > > > > > > To: gnats-bugs@gnats.NetBSD.org
 > > > > > > Cc: 
 > > > > > > Subject: PR/51485 CVS commit: pkgsrc/www/seamonkey
 > > > > > > Date: Sun, 18 Sep 2016 11:09:42 +0000
 > > > > > > 
 > > > > > >  Module Name:	pkgsrc
 > > > > > >  Committed By:	wiz
 > > > > > >  Date:		Sun Sep 18 11:09:42 UTC 2016
 > > > > > >  
 > > > > > >  Modified Files:
 > > > > > >  	pkgsrc/www/seamonkey: distinfo
 > > > > > >  Removed Files:
 > > > > > >  	pkgsrc/www/seamonkey/patches: patch-mozilla_gfx_ycbcr_yuv__row__arm.S
 > > > > > >  
 > > > > > >  Log Message:
 > > > > > >  Remove patch that adds a .s file as .S file.
 > > > > > >  
 > > > > > >  I can't believe both copies are needed.
 > > > > > >  
 > > > > > >  Addresses "pkgsrc cannot be checked out on a case-insensitive file system"
 > > > > > >  by Jonathan Schleifer, PR 51485.
 > > > > > 
 > > > > > This is wrong. The .s version doesn't work as is because it won't get
 > > > > > preprocessed first.
 > > > > 
 > > > > So we only need the .S patch and can remove the .s one?
 > > > 
 > > > Well, the .S patch is opaque and almost impossible to understand...
 > > 
 > > I think the .S patch is just a copy of the patched .s file.
 > 
 > Exactly. So it is impossible to determine from the copying patch, what
 > actually changed in the file. As such, it can't be maintained properly
 > after updates.

 That's true. But even if I add back the patch under a non-conflicting
 name, the package still won't build on a case-insensitive file system.
 It just fixes the checkout.

 What's your suggested solution?
  Thomas

From: Joerg Sonnenberger <joerg@bec.de>
To: Thomas Klausner <wiz@NetBSD.org>
Cc: Joerg Sonnenberger <joerg@bec.de>,
	NetBSD bugtracking <gnats-bugs@NetBSD.org>
Subject: Re: PR/51485 CVS commit: pkgsrc/www/seamonkey
Date: Mon, 19 Sep 2016 23:23:15 +0200

 On Mon, Sep 19, 2016 at 10:37:01PM +0200, Thomas Klausner wrote:
 > That's true. But even if I add back the patch under a non-conflicting
 > name, the package still won't build on a case-insensitive file system.
 > It just fixes the checkout.

 Worst case, just make it a manual patch...

 Joerg

From: Thomas Klausner <wiz@NetBSD.org>
To: Joerg Sonnenberger <joerg@bec.de>
Cc: NetBSD bugtracking <gnats-bugs@NetBSD.org>
Subject: Re: PR/51485 CVS commit: pkgsrc/www/seamonkey
Date: Mon, 19 Sep 2016 23:50:05 +0200

 On Mon, Sep 19, 2016 at 11:23:15PM +0200, Joerg Sonnenberger wrote:
 > On Mon, Sep 19, 2016 at 10:37:01PM +0200, Thomas Klausner wrote:
 > > That's true. But even if I add back the patch under a non-conflicting
 > > name, the package still won't build on a case-insensitive file system.
 > > It just fixes the checkout.
 > 
 > Worst case, just make it a manual patch...

 I don't understand.

 During the build there are two files called foo.S and foo.s, whatever
 I call the patch or if I do it manually. How is that supposed to work
 on a case-insensitive file system?
  Thomas

From: Joerg Sonnenberger <joerg@bec.de>
To: gnats-bugs@NetBSD.org
Cc: pkg-manager@netbsd.org, gnats-admin@netbsd.org, pkgsrc-bugs@netbsd.org,
	js-pkgsrc@heap.zone
Subject: Re: PR/51485 CVS commit: pkgsrc/www/seamonkey
Date: Tue, 20 Sep 2016 00:25:36 +0200

 On Mon, Sep 19, 2016 at 09:55:01PM +0000, Thomas Klausner wrote:
 > The following reply was made to PR pkg/51485; it has been noted by GNATS.
 > 
 > From: Thomas Klausner <wiz@NetBSD.org>
 > To: Joerg Sonnenberger <joerg@bec.de>
 > Cc: NetBSD bugtracking <gnats-bugs@NetBSD.org>
 > Subject: Re: PR/51485 CVS commit: pkgsrc/www/seamonkey
 > Date: Mon, 19 Sep 2016 23:50:05 +0200
 > 
 >  On Mon, Sep 19, 2016 at 11:23:15PM +0200, Joerg Sonnenberger wrote:
 >  > On Mon, Sep 19, 2016 at 10:37:01PM +0200, Thomas Klausner wrote:
 >  > > That's true. But even if I add back the patch under a non-conflicting
 >  > > name, the package still won't build on a case-insensitive file system.
 >  > > It just fixes the checkout.
 >  > 
 >  > Worst case, just make it a manual patch...
 >  
 >  I don't understand.
 >  
 >  During the build there are two files called foo.S and foo.s, whatever
 >  I call the patch or if I do it manually. How is that supposed to work
 >  on a case-insensitive file system?

 Well, are we talking about checking out the pkgsrc tree or building
 things on an case-insensitive file system? They are completely different
 problems. As is, you have just broken the package for no good reason.

 Joerg

Responsible-Changed-From-To: pkg-manager->joerg
Responsible-Changed-By: wiz@NetBSD.org
Responsible-Changed-When: Mon, 19 Sep 2016 22:39:43 +0000
Responsible-Changed-Why:
joerg knows how to do this.


State-Changed-From-To: closed->open
State-Changed-By: wiz@NetBSD.org
State-Changed-When: Mon, 19 Sep 2016 22:39:43 +0000
State-Changed-Why:


From: Takahiro Kambe <taca@NetBSD.org>
To: joerg@NetBSD.org
Cc: gnats-bugs@NetBSD.org
Subject: Re: pkg/51485 (pkgsrc cannot be checked out on a case-insensitive
 file system)
Date: Mon, 10 Oct 2016 01:49:20 +0900 (JST)

 Hi,

 In message <20160919223943.4F18D7A285@mollari.NetBSD.org>
 	on Mon, 19 Sep 2016 22:39:43 +0000 (UTC),
 	wiz@NetBSD.org wrote:
 > Synopsis: pkgsrc cannot be checked out on a case-insensitive file system
 > 
 > Responsible-Changed-From-To: pkg-manager->joerg
 > Responsible-Changed-By: wiz@NetBSD.org
 > Responsible-Changed-When: Mon, 19 Sep 2016 22:39:43 +0000
 > Responsible-Changed-Why:
 > joerg knows how to do this.
 What is the current status?

 It is very irritated problem to update latest pkgsrc tree on
 case-insensitive.

 -- 
 Takahiro Kambe <taca@NetBSD.org>

From: "J. Lewis Muir" <jlmuir@imca-cat.org>
To: gnats-bugs@NetBSD.org
Cc: 
Subject: Re: pkg/51485
Date: Sat, 29 Oct 2016 00:43:35 -0500

 On Tue, 20 Sep 2016 00:25:36 +0200, Joerg Sonnenberger wrote:
 > Well, are we talking about checking out the pkgsrc tree or building
 > things on an case-insensitive file system? They are completely different
 > problems. As is, you have just broken the package for no good reason.

 I'm interested in the problem of checking out the pkgsrc tree.  On
 Darwin (macOS Sierra 10.12.1), immediately after a clone or checkout,
 "git status" reports the following:

   modified:   www/seamonkey/patches/patch-mozilla_gfx_ycbcr_yuv__row__arm.s

 I don't intend to build that package, but it messes up my work because
 Git thinks I've modified a file that I haven't.

 If one of the conflicting .s or .S files could be renamed to something
 that differs in more than just case (e.g., rename the .s patch file to
 .s_), then it would fix the checking-out problem and make the building
 problem no worse than it currently is.

 Thanks!

 Lewis

From: "J. Lewis Muir" <jlmuir@imca-cat.org>
To: gnats-bugs@NetBSD.org
Cc: 
Subject: Re: pkg/51485
Date: Thu, 3 Nov 2016 16:55:50 -0500

 I have the same problem using CVS.  For example, a checkout results in a
 conflict:

   $ cvs -z2 -d anoncvs@anoncvs.NetBSD.org:/cvsroot checkout -P pkgsrc/www/seamonkey/patches
   [snip]
   U pkgsrc/www/seamonkey/patches/patch-mozilla_gfx_ycbcr_yuv__row__arm.S
   cvs checkout: move away `pkgsrc/www/seamonkey/patches/patch-mozilla_gfx_ycbcr_yuv__row__arm.s'; it is in the way
   C pkgsrc/www/seamonkey/patches/patch-mozilla_gfx_ycbcr_yuv__row__arm.s
   [snip]

 And every time I check the status, I get an indication that the .s file
 was updated:

   $ cd pkgsrc
   $ cvs -n update
   cvs update: Updating .
   cvs update: Updating www
   cvs update: Updating www/seamonkey
   cvs update: Updating www/seamonkey/patches
   U www/seamonkey/patches/patch-mozilla_gfx_ycbcr_yuv__row__arm.s

 Thanks!

 Lewis

State-Changed-From-To: open->feedback
State-Changed-By: dholland@NetBSD.org
State-Changed-When: Sun, 20 Nov 2016 00:43:11 +0000
State-Changed-Why:
looks like joerg fixed it on nov. 3 with -r1.139 of distinfo and corresponding
changes to the patches.


From: Jonathan Schleifer <js-pkgsrc@heap.zone>
To: gnats-bugs@NetBSD.org
Cc: 
Subject: Re: pkg/51485 (pkgsrc cannot be checked out on a case-insensitive
 file system)
Date: Sun, 20 Nov 2016 11:21:39 +0100

 WFM now.

State-Changed-From-To: feedback->closed
State-Changed-By: maya@NetBSD.org
State-Changed-When: Sun, 20 Nov 2016 12:30:51 +0000
State-Changed-Why:
feedback received, thanks


>Unformatted:

Home	PR Database Search