Re: Read-only /usr/obj/ no longer kosher?

2015-08-27 Thread Garrett Cooper

 On Aug 26, 2015, at 19:03, O'Connor, Daniel dar...@dons.net.au wrote:
 
 
 On 27 Aug 2015, at 08:25, Pawel Jakub Dawidek p...@freebsd.org wrote:
 On Tue, Aug 25, 2015 at 03:32:35PM -0700, NGie Cooper wrote:
 On Tue, Aug 25, 2015 at 3:21 PM, Xin Li delp...@delphij.net wrote:
 On 08/25/15 14:55, Pawel Jakub Dawidek wrote:
 Now that I think of it, it might have been that I did
 buildworld/buildkernel before -p1. Then freebsd-update updated
 newvers.sh and then I was trying to do installworld.
 
 Yes, I can now reproduce it with source updated to -p2.
 
 Yes, that's because freebsd-version.sh is generated from the files (but
 it's not clear to me whether if it's a bug or a feature that 'make
 install' checks if it's up-to-date and decides to regenerate it...).
 
 It's a quirk for sure. If you change the behavior, people will
 definitely complain as they will now need to go back and rebuild
 everything.
 
 What we have now is misleading. People should recompile. It is rather
 rare to see security advisory which bumps only patch level and something
 that doesn't require recompilation (eg. a shell script). Current
 behaviour would make people think they are running latest patch level
 because freebsd-version says so, eventhough they only did 'make
 installworld' without rebuilding affected binaries.
 
 So..
 How hard would it be to force CC/CXX to /usr/bin/false during installworld?

Trivial in FreeBSD. Just make it so in Makefile for the installworld target, 
add false to itools, and add appropriate special casing in bsd.compiler.mk.

Doing this just prevents recompiling though, so not pjd's case.

Also, this might break someone's random usecase where they need CC/CXX to do 
something meaningful at install time, however, the likelihood of it being 
correct is slim IMHO..
___
freebsd-current@freebsd.org mailing list
https://lists.freebsd.org/mailman/listinfo/freebsd-current
To unsubscribe, send any mail to freebsd-current-unsubscr...@freebsd.org


Re: Read-only /usr/obj/ no longer kosher?

2015-08-26 Thread Pawel Jakub Dawidek
On Tue, Aug 25, 2015 at 03:32:35PM -0700, NGie Cooper wrote:
 On Tue, Aug 25, 2015 at 3:21 PM, Xin Li delp...@delphij.net wrote:
  On 08/25/15 14:55, Pawel Jakub Dawidek wrote:
  Now that I think of it, it might have been that I did
  buildworld/buildkernel before -p1. Then freebsd-update updated
  newvers.sh and then I was trying to do installworld.
 
  Yes, I can now reproduce it with source updated to -p2.
 
  Yes, that's because freebsd-version.sh is generated from the files (but
  it's not clear to me whether if it's a bug or a feature that 'make
  install' checks if it's up-to-date and decides to regenerate it...).
 
 It's a quirk for sure. If you change the behavior, people will
 definitely complain as they will now need to go back and rebuild
 everything.

What we have now is misleading. People should recompile. It is rather
rare to see security advisory which bumps only patch level and something
that doesn't require recompilation (eg. a shell script). Current
behaviour would make people think they are running latest patch level
because freebsd-version says so, eventhough they only did 'make
installworld' without rebuilding affected binaries.

-- 
Pawel Jakub Dawidek   http://www.wheelsystems.com
FreeBSD committer http://www.FreeBSD.org
Am I Evil? Yes, I Am! http://mobter.com


pgpRsLnByGkaA.pgp
Description: PGP signature


Re: Read-only /usr/obj/ no longer kosher?

2015-08-26 Thread O'Connor, Daniel

 On 27 Aug 2015, at 08:25, Pawel Jakub Dawidek p...@freebsd.org wrote:
 On Tue, Aug 25, 2015 at 03:32:35PM -0700, NGie Cooper wrote:
 On Tue, Aug 25, 2015 at 3:21 PM, Xin Li delp...@delphij.net wrote:
 On 08/25/15 14:55, Pawel Jakub Dawidek wrote:
 Now that I think of it, it might have been that I did
 buildworld/buildkernel before -p1. Then freebsd-update updated
 newvers.sh and then I was trying to do installworld.
 
 Yes, I can now reproduce it with source updated to -p2.
 
 Yes, that's because freebsd-version.sh is generated from the files (but
 it's not clear to me whether if it's a bug or a feature that 'make
 install' checks if it's up-to-date and decides to regenerate it...).
 
 It's a quirk for sure. If you change the behavior, people will
 definitely complain as they will now need to go back and rebuild
 everything.
 
 What we have now is misleading. People should recompile. It is rather
 rare to see security advisory which bumps only patch level and something
 that doesn't require recompilation (eg. a shell script). Current
 behaviour would make people think they are running latest patch level
 because freebsd-version says so, eventhough they only did 'make
 installworld' without rebuilding affected binaries.

So..
How hard would it be to force CC/CXX to /usr/bin/false during installworld?

--
Daniel O'Connor
The nice thing about standards is that there
are so many of them to choose from.
 -- Andrew Tanenbaum
GPG Fingerprint - 5596 B766 97C0 0E94 4347 295E E593 DC20 7B3F CE8C

___
freebsd-current@freebsd.org mailing list
https://lists.freebsd.org/mailman/listinfo/freebsd-current
To unsubscribe, send any mail to freebsd-current-unsubscr...@freebsd.org


Re: Read-only /usr/obj/ no longer kosher?

2015-08-25 Thread Pawel Jakub Dawidek
On Tue, Aug 25, 2015 at 11:04:37PM +0200, Pawel Jakub Dawidek wrote:
 On Sun, Aug 23, 2015 at 03:29:01PM -0700, Xin Li wrote:
  
  
  On 8/23/15 14:55, Pawel Jakub Dawidek wrote:
   I used to build world and kernel on one machine and export both /usr/src/ 
   and
   /usr/obj read-only to other machines. It doesn't work anymore (this is 
   from
   'make installworld'):
   
   === bin/freebsd-version (install)
   eval $(egrep '^(TYPE|REVISION|BRANCH)=' 
   /usr/src/bin/freebsd-version/../../sys/conf/newvers.sh) ;  if ! sed -e  
   s/@@TYPE@@/${TYPE}/g;  s/@@REVISION@@/${REVISION}/g;  
   s/@@BRANCH@@/${BRANCH}/g;   
   /usr/src/bin/freebsd-version/freebsd-version.sh.in freebsd-version.sh ; 
   then  rm -f freebsd-version.sh ;  exit 1 ;  fi
   cannot create freebsd-version.sh: Permission denied
   rm: freebsd-version.sh: Read-only file system
   *** Error code 1
  
  What's the modification times of
  /usr/obj/usr/bin/freebsd-version/freebsd-version.sh,
  /usr/src/bin/freebsd-version/freebsd-version.sh and
  /usr/src/sys/conf/newvers.sh?
 
 I saw it twice, but cannot reproduce it anymore. This is 10.2-RELEASE,
 I've send it to current@ by mistake. All in all my expectation is that
 we shouldn't modify obj/ during installworld.

Now that I think of it, it might have been that I did
buildworld/buildkernel before -p1. Then freebsd-update updated
newvers.sh and then I was trying to do installworld.

-- 
Pawel Jakub Dawidek   http://www.wheelsystems.com
FreeBSD committer http://www.FreeBSD.org
Am I Evil? Yes, I Am! http://mobter.com
___
freebsd-current@freebsd.org mailing list
https://lists.freebsd.org/mailman/listinfo/freebsd-current
To unsubscribe, send any mail to freebsd-current-unsubscr...@freebsd.org


Re: Read-only /usr/obj/ no longer kosher?

2015-08-25 Thread Pawel Jakub Dawidek
On Tue, Aug 25, 2015 at 11:53:47PM +0200, Pawel Jakub Dawidek wrote:
 On Tue, Aug 25, 2015 at 11:04:37PM +0200, Pawel Jakub Dawidek wrote:
  On Sun, Aug 23, 2015 at 03:29:01PM -0700, Xin Li wrote:
   
   
   On 8/23/15 14:55, Pawel Jakub Dawidek wrote:
I used to build world and kernel on one machine and export both 
/usr/src/ and
/usr/obj read-only to other machines. It doesn't work anymore (this is 
from
'make installworld'):

=== bin/freebsd-version (install)
eval $(egrep '^(TYPE|REVISION|BRANCH)=' 
/usr/src/bin/freebsd-version/../../sys/conf/newvers.sh) ;  if ! sed -e 
 s/@@TYPE@@/${TYPE}/g;  s/@@REVISION@@/${REVISION}/g;  
s/@@BRANCH@@/${BRANCH}/g;   
/usr/src/bin/freebsd-version/freebsd-version.sh.in freebsd-version.sh 
; then  rm -f freebsd-version.sh ;  exit 1 ;  fi
cannot create freebsd-version.sh: Permission denied
rm: freebsd-version.sh: Read-only file system
*** Error code 1
   
   What's the modification times of
   /usr/obj/usr/bin/freebsd-version/freebsd-version.sh,
   /usr/src/bin/freebsd-version/freebsd-version.sh and
   /usr/src/sys/conf/newvers.sh?
  
  I saw it twice, but cannot reproduce it anymore. This is 10.2-RELEASE,
  I've send it to current@ by mistake. All in all my expectation is that
  we shouldn't modify obj/ during installworld.
 
 Now that I think of it, it might have been that I did
 buildworld/buildkernel before -p1. Then freebsd-update updated
 newvers.sh and then I was trying to do installworld.

Yes, I can now reproduce it with source updated to -p2.

-- 
Pawel Jakub Dawidek   http://www.wheelsystems.com
FreeBSD committer http://www.FreeBSD.org
Am I Evil? Yes, I Am! http://mobter.com
___
freebsd-current@freebsd.org mailing list
https://lists.freebsd.org/mailman/listinfo/freebsd-current
To unsubscribe, send any mail to freebsd-current-unsubscr...@freebsd.org


Re: Read-only /usr/obj/ no longer kosher?

2015-08-25 Thread Xin Li
On 08/25/15 14:55, Pawel Jakub Dawidek wrote:
 Now that I think of it, it might have been that I did
 buildworld/buildkernel before -p1. Then freebsd-update updated
 newvers.sh and then I was trying to do installworld.
 
 Yes, I can now reproduce it with source updated to -p2.

Yes, that's because freebsd-version.sh is generated from the files (but
it's not clear to me whether if it's a bug or a feature that 'make
install' checks if it's up-to-date and decides to regenerate it...).

Cheers,
-- 
Xin LI delp...@delphij.nethttps://www.delphij.net/
FreeBSD - The Power to Serve!   Live free or die



signature.asc
Description: OpenPGP digital signature


Re: Read-only /usr/obj/ no longer kosher?

2015-08-25 Thread NGie Cooper
On Tue, Aug 25, 2015 at 3:21 PM, Xin Li delp...@delphij.net wrote:
 On 08/25/15 14:55, Pawel Jakub Dawidek wrote:
 Now that I think of it, it might have been that I did
 buildworld/buildkernel before -p1. Then freebsd-update updated
 newvers.sh and then I was trying to do installworld.

 Yes, I can now reproduce it with source updated to -p2.

 Yes, that's because freebsd-version.sh is generated from the files (but
 it's not clear to me whether if it's a bug or a feature that 'make
 install' checks if it's up-to-date and decides to regenerate it...).

It's a quirk for sure. If you change the behavior, people will
definitely complain as they will now need to go back and rebuild
everything.

By and large though, recompiling things is really bad. That's I've
seen others do `CC=false CXX=false` when calling installworld.
___
freebsd-current@freebsd.org mailing list
https://lists.freebsd.org/mailman/listinfo/freebsd-current
To unsubscribe, send any mail to freebsd-current-unsubscr...@freebsd.org


Re: Read-only /usr/obj/ no longer kosher?

2015-08-25 Thread Pawel Jakub Dawidek
On Sun, Aug 23, 2015 at 03:29:01PM -0700, Xin Li wrote:
 
 
 On 8/23/15 14:55, Pawel Jakub Dawidek wrote:
  I used to build world and kernel on one machine and export both /usr/src/ 
  and
  /usr/obj read-only to other machines. It doesn't work anymore (this is from
  'make installworld'):
  
  === bin/freebsd-version (install)
  eval $(egrep '^(TYPE|REVISION|BRANCH)=' 
  /usr/src/bin/freebsd-version/../../sys/conf/newvers.sh) ;  if ! sed -e  
  s/@@TYPE@@/${TYPE}/g;  s/@@REVISION@@/${REVISION}/g;  
  s/@@BRANCH@@/${BRANCH}/g;   
  /usr/src/bin/freebsd-version/freebsd-version.sh.in freebsd-version.sh ; 
  then  rm -f freebsd-version.sh ;  exit 1 ;  fi
  cannot create freebsd-version.sh: Permission denied
  rm: freebsd-version.sh: Read-only file system
  *** Error code 1
 
 What's the modification times of
 /usr/obj/usr/bin/freebsd-version/freebsd-version.sh,
 /usr/src/bin/freebsd-version/freebsd-version.sh and
 /usr/src/sys/conf/newvers.sh?

I saw it twice, but cannot reproduce it anymore. This is 10.2-RELEASE,
I've send it to current@ by mistake. All in all my expectation is that
we shouldn't modify obj/ during installworld.

-- 
Pawel Jakub Dawidek   http://www.wheelsystems.com
FreeBSD committer http://www.FreeBSD.org
Am I Evil? Yes, I Am! http://mobter.com
___
freebsd-current@freebsd.org mailing list
https://lists.freebsd.org/mailman/listinfo/freebsd-current
To unsubscribe, send any mail to freebsd-current-unsubscr...@freebsd.org


Re: Read-only /usr/obj/ no longer kosher?

2015-08-23 Thread Xin Li


On 8/23/15 14:55, Pawel Jakub Dawidek wrote:
 I used to build world and kernel on one machine and export both /usr/src/ and
 /usr/obj read-only to other machines. It doesn't work anymore (this is from
 'make installworld'):
 
 === bin/freebsd-version (install)
 eval $(egrep '^(TYPE|REVISION|BRANCH)=' 
 /usr/src/bin/freebsd-version/../../sys/conf/newvers.sh) ;  if ! sed -e  
 s/@@TYPE@@/${TYPE}/g;  s/@@REVISION@@/${REVISION}/g;  
 s/@@BRANCH@@/${BRANCH}/g;   
 /usr/src/bin/freebsd-version/freebsd-version.sh.in freebsd-version.sh ; then 
  rm -f freebsd-version.sh ;  exit 1 ;  fi
 cannot create freebsd-version.sh: Permission denied
 rm: freebsd-version.sh: Read-only file system
 *** Error code 1

What's the modification times of
/usr/obj/usr/bin/freebsd-version/freebsd-version.sh,
/usr/src/bin/freebsd-version/freebsd-version.sh and
/usr/src/sys/conf/newvers.sh?

Cheers,



signature.asc
Description: OpenPGP digital signature