Skip site navigation (1)Skip section navigation (2)
Date:      Wed, 17 Jul 2013 23:47:33 +0100 (BST)
From:      Anton Shterenlikht <mexas@bris.ac.uk>
To:        mexas@bristol.ac.uk, swhetzel@gmail.com
Cc:        freebsd-ports@freebsd.org
Subject:   Re: svn diff/patch change file permission in cad/feappv
Message-ID:  <201307172247.r6HMlXbc025421@mech-cluster241.men.bris.ac.uk>
In-Reply-To: <CACdU%2Bf8m=1GZN2MM4tGRQ2208UDZOOZL0mpACVv6TPfx3RdDVw@mail.gmail.com>

next in thread | previous in thread | raw e-mail | index | archive | help
>From swhetzel@gmail.com Wed Jul 17 23:41:31 2013
>On Wed, Jul 17, 2013 at 5:26 PM, Anton Shterenlikht <mexas@bris.ac.uk> wrote:
>> >From swhetzel@gmail.com Wed Jul 17 23:20:37 2013
>>>On Wed, Jul 17, 2013 at 4:48 PM, Anton Shterenlikht <mexas@bris.ac.uk> wrote:
>>>> >From swhetzel@gmail.com Wed Jul 17 22:44:56 2013
>>>>>
>>>>>On Wed, Jul 17, 2013 at 9:50 AM, Anton Shterenlikht <mexas@bris.ac.uk> wrote:
>>>>>> I'm updating cad/feappv.
>>>>>> I noticed that svn diff, followed by svn patch
>>>>>> changes file permissions for 2 files out of 3.
>>>>>> Please help
>>>>>>
>>>>>> After I finished updating the port, I did:
>>>>>>
>>>>>> # pwd
>>>>>> /usr/ports/cad/feappv
>>>>>> # ls -al
>>>>>> total 24
>>>>>> drwxr-xr-x    2 root  wheel   512 Jul 17 15:45 .
>>>>>> drwxr-xr-x  108 root  wheel  2560 Mar 22 03:45 ..
>>>>>> -rw-r--r--    1 root  wheel  1541 Jul 17 15:41 Makefile
>>>>>> -rw-r--r--    1 root  wheel   268 Jul 17 15:38 distinfo
>>>>>> -rw-r--r--    1 root  wheel   473 Jul 17 15:39 pkg-descr
>>>>>> # svn diff > feappv.diff
>>>>>> # svn revert *
>>>>>> Reverted 'Makefile'
>>>>>> Reverted 'distinfo'
>>>>>> Skipped 'feappv.diff'
>>>>>> Reverted 'pkg-descr'
>>>>>> # ls -al
>>>>>> total 32
>>>>>> drwxr-xr-x    2 root  wheel   512 Jul 17 15:46 .
>>>>>> drwxr-xr-x  108 root  wheel  2560 Mar 22 03:45 ..
>>>>>> -rw-r--r--    1 root  wheel  1613 Jul 17 15:46 Makefile
>>>>>> -rw-r--r--    1 root  wheel   268 Jul 17 15:46 distinfo
>>>>>> -rw-r--r--    1 root  wheel  2439 Jul 17 15:46 feappv.diff
>>>>>> -rw-r--r--    1 root  wheel   474 Jul 17 15:46 pkg-descr
>>>>>> # svn patch feappv.diff
>>>>>> U         Makefile
>>>>>> U         distinfo
>>>>>> U         pkg-descr
>>>>>> # ls -al
>>>>>> total 32
>>>>>> drwxr-xr-x    2 root  wheel   512 Jul 17 15:46 .
>>>>>> drwxr-xr-x  108 root  wheel  2560 Mar 22 03:45 ..
>>>>>> -rw-------    1 root  wheel  1541 Jul 17 15:46 Makefile
>>>>>> -rw-------    1 root  wheel   268 Jul 17 15:46 distinfo
>>>>>> -rw-r--r--    1 root  wheel  2439 Jul 17 15:46 feappv.diff
>>>>>> -rw-r--r--    1 root  wheel   473 Jul 17 15:46 pkg-descr
>>>>>> #
>>>>>>
>>>>>> Note that now Makefile and distinfo have 600 perms,
>>>>>> instead of 644. Have I missed some svn diff/patch
>>>>>> option? Or is there a reason for the change in permissions?
>>>>>>
>>>>>
>>>>>Do you have umask set in your environment?  As a umask  of 077 would
>>>>>explain the issue.
>>>>
>>>> no, no umask anywhere.
>>>>
>>>> Still, why would only 2 files out of 3 change permissions?
>>>
>>>My best guess is that feappv.diff patch is being applied to only patch
>>>Makefile and distinfo.  pkg-descr is not being touched by the patch.
>>>That is why I suspect that umask might be involved.
>>
>> sorry, I could've been more clear.
>> Yes, the patch touches 3 files: Makefile, distinfo, pkg-descr.
>> For some reason, Makefile and distinfo changed permission
>> after the patch is applied.
>>
>The only other theory I have is that when you had edited the Makefile
>and distinfo the umask was set to 077.  Which changed the file
>permissions.  Then you used svn diff  to create the feappv.diff patch.
> SVN may have recorded the file permission changes in the patch file.
>Is there anything extra in the patch file?
>
>Otherwise I'm out of ideals.

I think this unlikely. I never mess with umask at all.

The patch has nothing related to permissions:

http://www.freebsd.org/cgi/query-pr.cgi?pr=180611

Many thanks

Anton




Want to link to this message? Use this URL: <https://mail-archive.FreeBSD.org/cgi/mid.cgi?201307172247.r6HMlXbc025421>